Last modified: 2014-04-29 17:10:09 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T48612, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 46612 - Message key '(Filepage|Common|Handheld|Print|Monobook).css' does not appear to be a full key
Message key '(Filepage|Common|Handheld|Print|Monobook).css' does not appear t...
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
1.21.x
All All
: High major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: 46579
  Show dependency treegraph
 
Reported: 2013-03-27 18:48 UTC by Sam Reed (reedy)
Modified: 2014-04-29 17:10 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sam Reed (reedy) 2013-03-27 18:48:40 UTC
2013-03-27 10:21:49 mw1171 commonswiki: [4063a65f] /w/index.php?title=MediaWiki:Filepage.css&action=raw&maxage=2678400&usemsgcache=yes&ctype=text%2Fcss&smaxage=2678400   Exception from line 637 of /usr/local/apache/common-local/php-1.21wmf12/includes/cache/MessageCache.php: Message key 'Filepage.css' does not appear to be a full key.
#0 /usr/local/apache/common-local/php-1.21wmf12/includes/actions/RawAction.php(140): MessageCache->get('Filepage.css', true, true, true)
#1 /usr/local/apache/common-local/php-1.21wmf12/includes/actions/RawAction.php(101): RawAction->getRawText()
#2 /usr/local/apache/common-local/php-1.21wmf12/includes/Action.php(542): RawAction->onView()
#3 /usr/local/apache/common-local/php-1.21wmf12/includes/Wiki.php(439): FormlessAction->show()
#4 /usr/local/apache/common-local/php-1.21wmf12/includes/Wiki.php(305): MediaWiki->performAction(Object(Article), Object(Title))
#5 /usr/local/apache/common-local/php-1.21wmf12/includes/Wiki.php(565): MediaWiki->performRequest()
#6 /usr/local/apache/common-local/php-1.21wmf12/includes/Wiki.php(458): MediaWiki->main()
#7 /usr/local/apache/common-local/php-1.21wmf12/index.php(59): MediaWiki->run()
#8 /usr/local/apache/common-local/w/index.php(3): require('/usr/local/apac...')
#9 {main}




2013-03-27 18:43:47 mw1066 eswiki: [3bb04644] /w/index.php?title=MediaWiki:Common.css&usemsgcache=yes&ctype=text/css&smaxage=2678400&action=raw&maxage=2678400   Exception from line 637 of /usr/local/apache/common-local/php-1.21wmf12/includes/cache/MessageCache.php: Message key 'Common.css' does not appear to be a full key.
#0 /usr/local/apache/common-local/php-1.21wmf12/includes/actions/RawAction.php(140): MessageCache->get('Common.css', true, true, true)
#1 /usr/local/apache/common-local/php-1.21wmf12/includes/actions/RawAction.php(101): RawAction->getRawText()
#2 /usr/local/apache/common-local/php-1.21wmf12/includes/Action.php(542): RawAction->onView()
#3 /usr/local/apache/common-local/php-1.21wmf12/includes/Wiki.php(439): FormlessAction->show()
#4 /usr/local/apache/common-local/php-1.21wmf12/includes/Wiki.php(305): MediaWiki->performAction(Object(Article), Object(Title))
#5 /usr/local/apache/common-local/php-1.21wmf12/includes/Wiki.php(565): MediaWiki->performRequest()
#6 /usr/local/apache/common-local/php-1.21wmf12/includes/Wiki.php(458): MediaWiki->main()
#7 /usr/local/apache/common-local/php-1.21wmf12/index.php(59): MediaWiki->run()
#8 /usr/local/apache/common-local/w/index.php(3): require('/usr/local/apac...')
#9 {main}
Comment 1 Sam Reed (reedy) 2013-03-28 05:16:01 UTC
84   Exception from line 637 of /usr/local/apache/common-local/php-1.21wmf12/includes/cache/MessageCache.php: Message key 'Filepage.css' does not appear to be a full key.
2   Exception from line 637 of /usr/local/apache/common-local/php-1.21wmf12/includes/cache/MessageCache.php: Message key 'Handheld.css' does not appear to be a full key.
2   Exception from line 637 of /usr/local/apache/common-local/php-1.21wmf12/includes/cache/MessageCache.php: Message key 'Common.css' does not appear to be a full key.
1   Exception from line 637 of /usr/local/apache/common-local/php-1.21wmf12/includes/cache/MessageCache.php: Message key 'Print.css' does not appear to be a full key.
1   Exception from line 637 of /usr/local/apache/common-local/php-1.21wmf12/includes/cache/MessageCache.php: Message key 'Monobook.css' does not appear to be a full key.
Comment 2 Andre Klapper 2013-03-28 10:52:26 UTC
(In reply to comment #0)
> 2013-03-27 10:21:49 mw1171 commonswiki: [4063a65f]
> /w/index.php?title=MediaWiki:Filepage.
> css&action=raw&maxage=2678400&usemsgcache=yes&ctype=text%2Fcss&smaxage=267840
> 0

URL works when dropping &usemsgcache=yes or &action=raw .
Comment 3 Leslie Carr 2013-03-28 19:39:41 UTC
FYI this is a huge % of the 5xx's at WMF at the moment.
Comment 4 Greg Grossmeier 2013-03-28 20:43:58 UTC
https://gerrit.wikimedia.org/r/#/c/56519/
Comment 5 Ori Livneh 2013-03-28 20:56:06 UTC
I think the error in the call to MessageCache::singleton()->get() dates back to a year to commit d1abcf7e, but that it was finally exposed by Gerrit change Iaaf6ccebd.
Comment 6 Ori Livneh 2013-03-28 21:05:07 UTC
(In reply to comment #5)
> I think the error in the call to MessageCache::singleton()->get() dates back
> to
> a year to commit d1abcf7e, but that it was finally exposed by Gerrit change
> Iaaf6ccebd.

Sorry, to be more specific: the bug goes back to Iceb1243ae, which set $isFullKey to true but dropped the call to wfMessage( $key )->inContentLanguage();
Comment 7 Matt Walker 2013-03-28 21:51:57 UTC
I have some concerns about the patch; I suspect there was a reason we request this with a fully qualified key -- if we don't and it is it becomes very expensive to lookup.

Is it instead possible to change the skin request to be qualified? Or can we guarantee that all calls to that function will be non qualified?

Katie is going to be deploying the revert to Iaaf6ccebd shortly; so we have some more time to fix this.
Comment 8 Andre Klapper 2013-03-29 18:08:01 UTC
So the example given in comment 0 works for me again, likely because of Katie reverting Iaaf6ccebd?
http://commons.wikimedia.org/w/index.php?title=MediaWiki:Filepage.css&action=raw&maxage=2678400&usemsgcache=yes&ctype=text%2Fcss&smaxage=2678400
Comment 9 Matt Walker 2013-03-29 22:50:16 UTC
(In reply to comment #8)
> So the example given in comment 0 works for me again, likely because of Katie
> reverting Iaaf6ccebd?
Yep :) But we'll be pushing it again just as soon as we fix the language cache bug so this isn't going to go away any time soon.
Comment 10 Andre Klapper 2013-04-08 14:39:36 UTC
(In reply to comment #9 by Matt Walker)
> But we'll be pushing it again just as soon as we fix the language cache
> bug so this isn't going to go away any time soon.

Matt: Is there a specific bug ID to follow for the "language cache bug"?
Comment 11 Matt Walker 2013-04-08 16:58:27 UTC
Uh; good question. I don't know if Nikerabbit ever filed one -- I don't see it in bugzilla as a quick search. I'll send an email and ask.
Comment 12 Andre Klapper 2013-04-16 14:51:54 UTC
Niklas Laxström wrote:
> I haven't file a bug report. It was working as expected until we tried
> to make a difference.
Comment 13 Gerrit Notification Bot 2013-05-28 10:58:36 UTC
https://gerrit.wikimedia.org/r/56519 (Gerrit Change I9729b55eb4ffe8f577eb0dc2073c4923b4be595d) | change ABANDONED [by Ori.livneh]
Comment 14 Andre Klapper 2013-07-08 11:07:45 UTC
(In reply to comment #9 by Matt Walker)
> But we'll be pushing it again just as soon as we fix the language
> cache bug so this isn't going to go away any time soon.

I'd really love to see "the language cache bug" (if still existing) filed somewhere by somebody who understand the internals, and made block this bug report. Also because this is blocking bug 46579 (which I haven't seen myself yet though).

Niklas / Matt, maybe?
Comment 15 Andre Klapper 2014-03-15 23:27:21 UTC
(In reply to Matt Walker from comment #7)
> Katie is going to be deploying the revert to Iaaf6ccebd shortly; so we have
> some more time to fix this.

Matt / Niklas:
Has any progress been made here to provide a fix and then to revert the revert?
Comment 16 Niklas Laxström 2014-04-29 17:04:22 UTC
We haven't done anything as far as I know, but Tyler has been working on a patch at https://gerrit.wikimedia.org/r/#/c/72867/ - I haven't checked if it addresses this bug as well.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links