User tests: Successful: Unsuccessful:
...of the argument list
Yeah, getting you. We will have to rearrange argument and will need to change application of this function everywhere where it is using this argument.
Thanks for the info. Closing this PR.
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-01-29 12:37:23 |
Labels |
Added:
?
|
Sorry @gunjanpatel we are still discussing which is the best approach to solve this and the other occurrences of the same CS error.
We will notify you the final decision.
Thanks for trying to fix it!
Great!
Thank you.
This changes code as it makes
$idtag
optional which is likely not what we want.To fix it one can either reorder the arguments which would break BC, or we can make
$optKey
and$optText
requiried and add the default values back in the code with something likeBut then we loose the info about default values in IDEs.
So there is probably not a simple fix for this.