[Standards] undefined state in XEP-0050
kevin.smith at isode.com
Thu Feb 22 09:28:32 UTC 2018
> On 21 Feb 2018, at 18:50, Florian Schmaus <flo at geekplace.eu> wrote:
> On 06.08.2015 17:42, Goffi wrote:
>> there is a little issue with XEP-0050: in section 3.4 bullet 3, it's is
>> said that when the <actions/> element is present:
>> - The action "execute" is always allowed, and is equivalent to the
>> action "next".
>> - The "next" action is typically the "next" button or option in a
>> wizard. If <next/> is not contained by the <actions/>, it is disabled.
>> So if "next" action is disabled, execute is allowed but equivalent to a
>> disabled action.
>> I have had an issue which seems related to this confusion with SleekXMPP
>> and Gajim: if the next action is disabled in sleekXMPP, Gajim still show
>> the "execute" button, but a click on it result in an error, while the
>> "finish" button act as expected.
>> My guess is that "execute" should be equivalent to "complete" when
>> "next" is not possible (but what if "complete" is disabled too ?).
> I think it is a little design weakness of XEP-0050: Ad-Hoc Commands that
> we have 'next' and 'execute'. It appears the whole protocol would be
> much simpler and less confusing if we just had 'execute’.
Execute is convenient - it’s not an action of its own per se, it’s just saying which of the provided actions is used by default (e.g. will be run when the user hits Enter), but allowing it to be sent as an action= value could have been better thought-out, I suspect (even then, it’s useful for the first stage of the form). So I suspect the underlying issue described here is UIs wrongly showing independent buttons for Execute and whatever Execute should mean.
> But this is unlikely to change ever. So here is how I understand it:
> - 'execute' always gets you into the next stage, and iff 'next' is an
> allowed action, then 'execute' is equivalent to 'next', or otherwise it
> is equivalent to 'complete’.
I think that if we follow what 50 says, we actually reach the conclusion that if execute isn’t set on actions, it’s equivalent to next, which may be disabled. Which basically means that an actions list not including next and not including an execute value is saying “The default is next, and it’s disabled”, which is more or less a broken xep 50 command so the responder shouldn’t send it. Changing to have the default switch to complete if there’s no next is probably not harmful, although is a change in behaviour from what we have.
I suggest the most helpful thing to do here is to add some additional text explaining that Execute shouldn’t be a button of its own in the UI, but is an indication of default, if people are misunderstanding that. If we do that, the point of changing default in the absence of next is less important, but I’m not opposed (it is a change in behaviour, though, so needs Council vote IMO).
More information about the Standards