Page MenuHomePhabricator

Flow: choosing Permalink causes exception "Input UUID strings must be lowercase"
Closed, ResolvedPublic

Description

As a logged in user at http://en.wikipedia.beta.wmflabs.org/wiki/Talk:Flow_QA

for any topic

choose Permalink

result is a stack trace in the browser:

Invalid value was provided for loading flow content.

Return to Main Page

[e94f879c] /wiki/Topic:S493t6evn7osywnm Exception from line 101 of /srv/mediawiki/php-master/extensions/Flow/includes/Model/UUID.php: Input UUID strings must be lowercase

Backtrace:

#0 /srv/mediawiki/php-master/extensions/Flow/includes/Model/UUID.php(180): Flow\Model\UUID->__construct(string, string)
#1 /srv/mediawiki/php-master/extensions/Flow/Hooks.php(887): Flow\Model\UUID::create(string)
#2 /srv/mediawiki/php-master/includes/Hooks.php(206): FlowHooks::onTitleSquidURLs(Title, array)
#3 /srv/mediawiki/php-master/includes/GlobalFunctions.php(4004): Hooks::run(string, array, NULL)
#4 /srv/mediawiki/php-master/includes/Title.php(3562): wfRunHooks(string, array)
#5 /srv/mediawiki/php-master/includes/MediaWiki.php(409): Title->getSquidURLs()
#6 /srv/mediawiki/php-master/includes/MediaWiki.php(282): MediaWiki->performAction(Article, Title)
#7 /srv/mediawiki/php-master/includes/MediaWiki.php(584): MediaWiki->performRequest()
#8 /srv/mediawiki/php-master/includes/MediaWiki.php(435): MediaWiki->main()
#9 /srv/mediawiki/php-master/index.php(46): MediaWiki->run()
#10 /srv/mediawiki/w/index.php(3): include(string)
#11 {main}


Version: unspecified
Severity: major

Details

Reference
bz72086

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:56 AM
bzimport set Reference to bz72086.
bzimport added a subscriber: Unknown Object (MLST).

This doesn't seem to happen on ee-flow, running the same Flow "tip of master" code.

gerritadmin wrote:

Change 166873 had a related patch set uploaded by EBernhardson:
Bug: UUIDs should always be lowercase

https://gerrit.wikimedia.org/r/166873

gerritadmin wrote:

Change 166873 abandoned by EBernhardson:
Bug: UUIDs should always be lowercase

Reason:
should not be against a deploy branch

https://gerrit.wikimedia.org/r/166873

gerritadmin wrote:

Change 166875 had a related patch set uploaded by EBernhardson:
Bug: UUIDs should always be lowercase

https://gerrit.wikimedia.org/r/166875

gerritadmin wrote:

Change 166875 merged by jenkins-bot:
Bug: UUIDs should always be lowercase

https://gerrit.wikimedia.org/r/166875

Seems fixed on beta labs.

Quiddity removed a subscriber: Maryana.Dec 19 2014, 1:36 AM