Last modified: 2014-01-03 16:14:07 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 T23325, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 21325 - If login required to read, IE7 throws JS error when not logged in
If login required to read, IE7 throws JS error when not logged in
Status: RESOLVED WORKSFORME
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.15.x
All All
: Low trivial (vote)
: ---
Assigned To: Nobody - You can work on this!
: testme
Depends on: 12247
Blocks:
  Show dependency treegraph
 
Reported: 2009-10-28 09:53 UTC by Cynthia
Modified: 2014-01-03 16:14 UTC (History)
3 users (show)

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


Attachments

Description Cynthia 2009-10-28 09:53:43 UTC
If $wgGroupPermissions['*']['read'] is set to 'false', then the call to /wiki/index.php?title=-&action=raw&smaxage=0&gen=js&useskin=myskin outputs not a javascript file, nor nothing, but the quite normal HTML page with the message about not being logged in. IE7 says this is not valid JS, and it does have a point there. This problem probably also exists for IE6 and IE8.

On my local installation, since we have a project-specific default skin, I patched it like so:

if($this->data['loggedin'	] == '') {
 $this->data['jsvarurl'  ] = '';
}

But a better solution would be setting jsvarurl to nothing if the user must log in to read before this point, or just outputting nothing for action=raw.
Comment 1 Andre Klapper 2012-10-16 16:55:09 UTC
Hi Cynthia! Sorry that nobody has taken a look at this report yet and given feedback.

Cynthia: If you have time, could you please update to a recent supported version (see http://www.mediawiki.org/wiki/Version_lifecycle ) and tell us if this problem still happens?
If this still happens, please add a comment to this report and tell us your exact software version in case that you know (by updating the "Version" field of this bug report).

If the problem does not happen anymore at all, please set the status of this report to RESOLVED > WORKSFORME. Thanks a lot for your help!
Comment 2 Andre Klapper 2013-01-29 21:39:16 UTC
I get a syntax error in Firefox Error Console. Confirming.
Comment 3 Umherirrender 2013-07-18 16:13:24 UTC
MySkin is (now) removed from MediaWiki.
Does this occurs in other skins?
Comment 4 Andre Klapper 2013-08-13 13:12:00 UTC
Cynthia: Closing as WORKSFORME for the time being, but if this still occurs in other skins, please add a comment here and reopen. Thanks!

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


Navigation
Links