Last modified: 2012-11-08 05:41:35 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 T43760, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 41760 - Minor edit checkbox seems to be ticked as default sometimes!
Minor edit checkbox seems to be ticked as default sometimes!
Status: RESOLVED INVALID
Product: MediaWiki
Classification: Unclassified
Page editing (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-11-04 08:59 UTC by wiki.pesky
Modified: 2012-11-08 05:41 UTC (History)
4 users (show)

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


Attachments

Description wiki.pesky 2012-11-04 08:59:33 UTC
Several of my recent (non-minor) edits are being marked as minor (because I don't notice that the checkbox is ticked before I hit save ...).  It would appear that the checkbox seems to be pre-ticked when the previous edit I made to that page was a minor one.  Doesn't seem to be pre-ticked the first time I edit a page, or when unticked on the previous edit.  This makes it appear that I'm marking non-muinor edits as minor deliberately - which is not a good thing! Can someone please fix/investigate?
Comment 1 Liangent 2012-11-04 09:06:11 UTC
Most likely you checked "Mark all edits minor by default" in your preferences. If you can't find that item now try to reset your preferences ([[Special:Preferences/reset]]).
Comment 2 wiki.pesky 2012-11-04 11:22:10 UTC
Oh!  Can't someone just make it possible to reset just that one preference?  I looked for it, but it wasn't there - and I certainly don't want to reset ALL my preferences!  Also, it's not marking all of them as minor by default - only those ones where the previous edit was marked as minor.
Comment 3 Liangent 2012-11-04 11:39:03 UTC
(In reply to comment #2)
> Oh!  Can't someone just make it possible to reset just that one preference?  I
> looked for it, but it wasn't there - and I certainly don't want to reset ALL my
> preferences!

But to reset only one, you have to tell the system which preference you want to reset -- but if you can find that preference you can simply flip its setting...

To help you find and confirm it: it should be located in the "Advanced options" section in "Editing" tab.

> Also, it's not marking all of them as minor by default - only
> those ones where the previous edit was marked as minor.

Is your browser pre-filling it from previously submitted form?
Comment 4 Jesús Martínez Novo (Ciencia Al Poder) 2012-11-04 11:53:58 UTC
On the english wikipedia that option isn't appearing in [[Special:Preferences]] for me, and it's not hidden by CSS or removed by JS.

That option is appearing using the API (search for minordefault):

http://en.wikipedia.org/w/api.php?action=query&meta=userinfo&uiprop=options&format=jsonfm

So you probably can change it with the API:

http://en.wikipedia.org/w/api.php?action=help&modules=options

If some options are "hidden" ( http://www.mediawiki.org/wiki/Manual:$wgHiddenPrefs ), they should probably do not apply at all instead of still being applied to users that changed them.

Note that some edits are not allowed to be minor, like new section creation or the creation of a new page. That would explain the "randomness".
Comment 5 Liangent 2012-11-04 12:07:18 UTC
(In reply to comment #4)
> On the english wikipedia that option isn't appearing in [[Special:Preferences]]
> for me, and it's not hidden by CSS or removed by JS.

Hmm it was turned off in bug 24313.
Comment 6 Liangent 2012-11-04 12:09:41 UTC
(In reply to comment #4)
> If some options are "hidden" (
> http://www.mediawiki.org/wiki/Manual:$wgHiddenPrefs ), they should probably do
> not apply at all instead of still being applied to users that changed them.

I guess this can be a valid bug. Should this bug be used to track this $wgHiddenPrefs issue, or create a new bug for it?
Comment 7 Brad Jorsch 2012-11-04 16:49:00 UTC
(In reply to comment #4)
> If some options are "hidden" (
> http://www.mediawiki.org/wiki/Manual:$wgHiddenPrefs ), they should probably do
> not apply at all instead of still being applied to users that changed them.

They don't. See includes/User.php, every pref that is hidden is automatically reset to the default when accessed in both getOption() and getOptions().

(In reply to comment #3)
> Is your browser pre-filling it from previously submitted form?

This is probably what's going on here. Or else they have a user script installed that remembers the setting of the checkbox and changes it.
Comment 8 wiki.pesky 2012-11-05 21:48:42 UTC
If the edit form has sections capable of being pre-filled by a browser, something should be put in place to prevent that from happening, ideally.  It shouldn't be a pre-fillable field.  Having said that, my browser doesn't pre-fill edit summaries, for example (though it does make suggestions, which I have, once or twice, inadvertently used just by not noticing that it had added bits I didn't intend to add!)

I don't think I have any user scripts which would check it on some occasions but not on others.
Comment 9 Liangent 2012-11-06 01:48:27 UTC
(In reply to comment #8)
> If the edit form has sections capable of being pre-filled by a browser,
> something should be put in place to prevent that from happening, ideally.  It
> shouldn't be a pre-fillable field.  Having said that, my browser doesn't
> pre-fill edit summaries, for example (though it does make suggestions, which I
> have, once or twice, inadvertently used just by not noticing that it had added
> bits I didn't intend to add!)
> 
> I don't think I have any user scripts which would check it on some occasions
> but not on others.

Can you confirm whether it's already ticked in HTML sent to you?
Comment 10 wiki.pesky 2012-11-06 09:17:17 UTC
I'm sorry, I don't quite understand which html you're referring to, and where; could you please clarify?
Comment 11 Liangent 2012-11-06 09:19:30 UTC
(In reply to comment #10)
> I'm sorry, I don't quite understand which html you're referring to, and where;
> could you please clarify?

Page source of the editing interface.
Comment 12 wiki.pesky 2012-11-06 14:54:06 UTC
I haz ""minordefault":0" in the html, but (as I suspected, on the page I checked on) the minor edit box was checked.  Is that helpful?
Comment 13 Brad Jorsch 2012-11-06 17:39:50 UTC
Depending on your browser, there should be a "View Source" or "View Page Source" somewhere; Google for "how to view page source in $BROWSER" for the appropriate value of $BROWSER.

Somewhere in all the HTML code you see, there should be something that resembles this:

  <input name="wpMinoredit" type="checkbox" value="1" tabindex="3" accesskey="i" id="wpMinoredit" />

What you're looking for begins with '<input', ends with '>', and has 'name="wpMinoredit"' in the middle. If you just copy and paste that tag in here, that will answer the question.
Comment 14 wiki.pesky 2012-11-06 18:53:29 UTC
Aha!  It has <input name="wpMinoredit" type="checkbox" value="1" tabindex="3" accesskey="i" id="wpMinoredit" />, and the minor edit box is ticked (I tried the same page I tried before, on which my previous edit had been marked as minor).

Aaaand ... on another page , it says <input name="wpMinoredit" type="checkbox" value="1" tabindex="3" accesskey="i" id="wpMinoredit" />, but the minor edit checkbox is NOT ticked. Hmm.  So what does that mean, then?
Comment 15 Brad Jorsch 2012-11-06 18:59:00 UTC
It means that the checkbox is not being served as checked in the HTML; either your browser is pre-filling it or some JavaScript on the page is checking it.

You could try removing all your user scripts and see if the problem goes away, then add them back one by one until it reappears.
Comment 16 wiki.pesky 2012-11-06 19:08:22 UTC
I can try (bummer, though ....) removing all my user scripts.  Why would a user script make it mark edits as minor ONLY on pages where the previous edit had been minor?
Comment 17 Brad Jorsch 2012-11-06 20:23:53 UTC
Because whoever wrote it thought that would be a feature?
Comment 18 Liangent 2012-11-07 05:24:05 UTC
Then unrelated to bug 24313...
Comment 19 Nemo 2012-11-07 05:52:54 UTC
Can you please tell us your user name?
Comment 20 wiki.pesky 2012-11-07 06:36:46 UTC
ThatPeskyCommoner (enwiki)
Comment 21 Nemo 2012-11-07 07:15:28 UTC
Ah, ok, I saw that one and its contributions but got lost on Jimbo-talk muds while looking for false minor edits in vain.
Then you must be using monobook and https://en.wikipedia.org/?diff=457283933&oldid=454508994 explains it, you're importing a ton of stuff for wikignomes including 

function Ohc_edit_summary1(){
    //Add a tag to the summary box
    setoptions(minor='true');         //removed ",watch='false'" in response to user notification 13 Nov. 2010
    setreason('[[User:Ohconfucius/script|script]]-assisted [[WP:MOS|style]] audit, incl. [[WP:MOSDATES|dates]], [[MOS:LINK]] and [[MOS:TEXT]]', 'append');
    doaction('diff');
}
 
function Ohc_edit_summary2(){
// Add a tag to the summary box
    setoptions(minor='true');         //removed ",watch='false'" in response to user notification 13 Nov. 2010
    setreason('[[User:Ohconfucius/script|script]]-assisted [[WP:MOS|style]] audit, incl. [[WP:ENGVAR|English variants]], [[WP:MOSDATES|dates]], [[MOS:LINK]] and [[MOS:TEXT]]', 'append');
    doaction('diff');
}

[the latter two times, on your monobook.js and in the script you import].
You have to remove all the stuff you're unsure of/you don't need, check if there are changes to other scripts you import etc. Nothing to do here on bugzilla.
Comment 22 wiki.pesky 2012-11-07 08:21:28 UTC
OK, many thanks for that.  I shall get around to that when I have some more spare moments (and meanwhile try to remember to visually double-check the box to see if it's appropriately ticked or not!)
Comment 23 wiki.pesky 2012-11-08 05:41:35 UTC
I've removed a ton of scripts, but it still seems to be doing it (I missed a couple this morning).  Any m ore ideas?

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


Navigation
Links