Item13217: Verify button not passing in empty elements.

pencil
Priority: Urgent
Current State: No Action Required
Released In: 1.2.0
Target Release: n/a
Applies To: Extension
Component: Configure
Branches:
Reported By: GeorgeClark
Waiting For:
Last Change By: GeorgeClark
Configure "Verify" button used on the paths elements fails to send in an empty string.

Enter /bin/view into {ScriptUrlPaths}{view} and press verify. The JSON sent is:
{"jsonrpc":"2.0","method":"wizard","params":{"wizard":"ScriptHash","method":"verify","keys":"{ScriptUrlPaths}{view}","set":{"{ScriptUrlPaths}{view}":"/bin"},"cfgusername":"testuser","cfgpassword":"xxxxxxx"},"id":"iCall-verify_15"}

"set":{"{ScriptUrlPaths}{view}":"/bin"} results in the "verify" script testing with the /bin path.

Now delete the field so that it is "empty" and press verify. The JSON sent is:

{"jsonrpc":"2.0","method":"wizard","params":{"wizard":"ScriptHash","method":"verify","keys":"{ScriptUrlPaths}{view}","set":{},"cfgusername":"testuser","cfgpassword":"xxxxxxx"},"id":"iCall-verify_17"}

The "set":{} result in the variable being ignored. The test is run using the value of {ScriptUrlPath} instead of the {ScriptUrlPaths}{view} override.

There is no way to verify the short URL configuration until it's actually saved in the configuration.

-- GeorgeClark - 16 Jan 2015

I just tested what you describe above and it works perfectly (both Chrome and FF tested)

I get "set":{"{ScriptUrlPaths}{view}"}:""} in the request, which is what I'd expect. It looks to me like there may be some browser issues....

-- CrawfordCurrie - 19 Jan 2015

Ah. I actually fixed this in distro:bd7a48212100d I forgot to check it in on this task. Query.pm Line 45

... no... never mind, that's not the fix. But indeed it's working perfectly now. Setting to No Action.

-- GeorgeClark - 19 Jan 2015
 

ItemTemplate edit

Summary Verify button not passing in empty elements.
ReportedBy GeorgeClark
Codebase
SVN Range
AppliesTo Extension
Component Configure
Priority Urgent
CurrentState No Action Required
WaitingFor
Checkins
TargetRelease n/a
ReleasedIn 1.2.0
CheckinsOnBranches
trunkCheckins
masterCheckins
ItemBranchCheckins
Release01x01Checkins
Topic revision: r3 - 19 Jan 2015, GeorgeClark
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy