BryanDavis has uploaded a new change for review. https://gerrit.wikimedia.org/r/92368
Change subject: Add clarification to wfUseMW() phpdoc ...................................................................... Add clarification to wfUseMW() phpdoc While reviewing Ia5a7661c I happened to stumble in to the dark labyrinth of PHP's version_compare() function. This comment attempts to leave bread crumbs behind for the next unwary adventurer who stumbles into the same damp cave. The tricky bit is that when '1.22wmf22' is compared with '1.22' internally it is as though you are comparing '1.22.wmf.22' with '1.22.0.0'. By the component-wise comparison then performed 'wmf' is considered the lowest priority value that could be in the third position. When using '1.22c' for the required version the 'c' is given the same lowest priority as 'wmf'. Change-Id: I58aeb8dc6b1ff31897d72eb44904d96487ad3653 --- M includes/GlobalFunctions.php 1 file changed, 8 insertions(+), 0 deletions(-) git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core refs/changes/68/92368/1 diff --git a/includes/GlobalFunctions.php b/includes/GlobalFunctions.php index 77c09e5..e3ef99a 100644 --- a/includes/GlobalFunctions.php +++ b/includes/GlobalFunctions.php @@ -3065,6 +3065,14 @@ * This is useful for extensions which due to their nature are not kept in sync * with releases * + * Note: Due to the behavior of PHP's version_compare() which is used in this + * fuction, if you want to allow the 'wmf' development versions add a 'c' (or + * any single letter other than 'a', 'b' or 'p') as a post-fix to your + * targeted version number. For example if you wanted to allow any variation + * of 1.22 use `wfUseMW( '1.22c' )`. Using an 'a' or 'b' instead of 'c' will + * result in the same comparison due to the internal logic of + * version_compare(). + * * @see perldoc -f use * * @param $req_ver Mixed: the version to check, can be a string, an integer, or -- To view, visit https://gerrit.wikimedia.org/r/92368 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I58aeb8dc6b1ff31897d72eb44904d96487ad3653 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: BryanDavis <[email protected]> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
