Last modified: 2014-02-12 23:32:43 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 T44592, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 42592 - User rights, preferences and other things are not saving in 1.20.1
User rights, preferences and other things are not saving in 1.20.1
Status: RESOLVED DUPLICATE of bug 42370
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
1.20.x
All All
: High blocker (vote)
: 1.20.x release
Assigned To: Chris Steipp
:
: 42622 42623 42643 (view as bug list)
Depends on: 42370
Blocks:
  Show dependency treegraph
 
Reported: 2012-12-01 04:24 UTC by Jeff Logan
Modified: 2014-02-12 23:32 UTC (History)
13 users (show)

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


Attachments
Workaround (601 bytes, patch)
2012-12-02 22:30 UTC, Zhen Lin
Details

Description Jeff Logan 2012-12-01 04:24:06 UTC
When attempting to change a user's group assignments with Special:UserRights, no changes are made on submission, it just reloads the page with the group permissions reset.
Comment 1 Alexander Mashin 2012-12-01 09:20:25 UTC
Actually, everything is broken in 1.20.1. Not only user rights are not saved, but user preferences and changes to watchlist are not recorded either.

This is not the first version with this issue. There was exactly the same bug in branch origin/REL1_20 (but not in tag 1.20).
Comment 2 Nemo 2012-12-01 18:36:53 UTC
Something that was reported earlier (by you, THO?) is that at least the logging is broken on WMF wikis as of now, see e.g. "and [?] (Was test)": 

(show/hide) 01:21, 1 December 2012 Steven (WMF) (talk | contribs | block) changed group membership for User:Steven (WMF) from importer, staff and transwiki importer to importer and (Was test)
(show/hide) 01:19, 1 December 2012 Steven (WMF) (talk | contribs | block) changed group membership for User:Steven (WMF) from importer and transwiki importer to importer, transwiki importer and staff (Test) 

at [[m:special:userrights/Steven (WMF)]].
Comment 3 Alex Monk 2012-12-01 18:43:35 UTC
(In reply to comment #2)
> Something that was reported earlier (by you, THO?) is that at least the logging
> is broken on WMF wikis as of now, see e.g. "and [?] (Was test)": 
> 
> (show/hide) 01:21, 1 December 2012 Steven (WMF) (talk | contribs | block)
> changed group membership for User:Steven (WMF) from importer, staff and
> transwiki importer to importer and (Was test)
> (show/hide) 01:19, 1 December 2012 Steven (WMF) (talk | contribs | block)
> changed group membership for User:Steven (WMF) from importer and transwiki
> importer to importer, transwiki importer and staff (Test) 
> 
> at [[m:special:userrights/Steven (WMF)]].

This is bug 42211, Gerrit change #33878.
Comment 4 Alexander Mashin 2012-12-02 08:11:30 UTC
And here is, I believe, the same bug: http://www.mail-archive.com/wikibugs-l@lists.wikimedia.org/msg177547.html.

I also overcame it by moving from branch origin/REL1_20 to tag 1.20.0, but it reappeared in 1.20.1.
Comment 5 Alex Monk 2012-12-02 11:46:01 UTC
*** Bug 42622 has been marked as a duplicate of this bug. ***
Comment 6 Alex Monk 2012-12-02 11:51:25 UTC
https://www.mediawiki.org/wiki/Project:Support_desk#User_preferences_won.27t_save_.28no_error.29_718
Scroll down to the 2nd December posts - Daniel Renfro mentions this error:
PHP Notice:  Undefined property: DatabaseMysql::$mTrxDoneWrites in ...includes/db/Database.php on line 512

Can someone confirm that this is bug 42370?
Comment 7 Alex Monk 2012-12-02 11:51:58 UTC
*** Bug 42623 has been marked as a duplicate of this bug. ***
Comment 8 Zhen Lin 2012-12-02 22:30:56 UTC
Created attachment 11451 [details]
Workaround

Here's a one-line fix. As far as I can tell the trouble is that $conn->writesOrCallbacksPending() always returns false, because mTrxDoneWrites is never set or incremented...
Comment 9 Andre Klapper 2012-12-03 10:20:25 UTC
*** Bug 42643 has been marked as a duplicate of this bug. ***
Comment 10 Andre Klapper 2012-12-03 10:23:25 UTC
Can somebody apply https://gerrit.wikimedia.org/r/#/c/36220/ to a 1.20.1 installation and confirm that this fixes the problem?
If not, testing the patch in comment 8 might also be worth a try.
Comment 11 Krinkle 2012-12-03 17:33:31 UTC
(In reply to comment #10)
> Can somebody apply https://gerrit.wikimedia.org/r/#/c/36220/ to a 1.20.1
> installation and confirm that this fixes the problem?

I can confirm this. I just updated a private wiki to 1.20.1 and ran into this. Applying that fixed it.

> If not, testing the patch in comment 8 might also be worth a try.

Replacing an important condition with a hardcoded "true" is not an acceptable patch, just for debugging.

This probably justifies a 1.20.2 release.
Comment 12 Andre Klapper 2012-12-03 18:32:01 UTC
(In reply to comment #11)
> (In reply to comment #10)
> > Can somebody apply https://gerrit.wikimedia.org/r/#/c/36220/ to 1.20.1
> I just updated to 1.20.1 and ran into this. Applying that fixed it.

Thanks, this makes it a duplicate of bug 42370.

*** This bug has been marked as a duplicate of bug 42370 ***

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


Navigation
Links