I ran into problems with unknown options, because array_search wasn't using the strict version, patch included.

CommentFileSizeAuthor
array_search_strict.patch3.83 KBattiks

Comments

mike stewart’s picture

mike stewart commented
Assigned: Unassigned » mike stewart

reviewing

mike stewart’s picture

mike stewart commented
Status: Needs review » Postponed (maintainer needs more info)

@attiks very awesome that you found a problem, a fix, and on TOP OF IT submitted a patch!

However, I'm not sure how to test? Hopefully it'll hit me before you need to reply, but what exactly is this fixing? I can tell from the patch this is fixing a search function, but I'm not certain what you might have been doing that was causing a problem? I can't think of how/where this is used, so not sure how to test.

attiks’s picture

attiks commented

@mike, no idea what the cause was, too long ago

mike stewart’s picture

mike stewart commented
Assigned: mike stewart » Unassigned

I'm removing assignment to me since I'm no longer maintaining the 7.x-2.x branch (its moved to DrupalPro Development Desktop), and I'm not sure of the future of Quickstart 2.x except that it wont be based on Ubuntu 12.04LTS (Unity)

MichaelCole’s picture

MichaelCole commented
Status: Postponed (maintainer needs more info) » Closed (fixed)

Hey Attiks, I'll email you directly :-)