Last modified: 2012-12-03 20:09:25 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 T44137, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 42137 - VisualEditor: Sync availability of "minor edit" with MW backend
VisualEditor: Sync availability of "minor edit" with MW backend
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: High trivial
: VE-deploy-2012-12-10
Assigned To: Krinkle
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2012-11-15 05:28 UTC by MZMcBride
Modified: 2012-12-03 20:09 UTC (History)
5 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description MZMcBride 2012-11-15 05:28:45 UTC 
Currently when using VisualEditor to create a new page, the "Save page" dialog box offers two checkboxes below the edit summary field: "This is a minor edit" and "Watch this page".

When creating a page, "This is a minor edit" does not typically show up. It's a bug that the checkbox appears when creating a page with the VisualEditor interface.

This bug is closely related to bug 40774.
Comment 1 Krinkle 2012-12-02 03:19:41 UTC 
The EditPage backends has this logic in three places:

* Getting html of checkboxes (only if user has minoredit right and if not on a  new page)
* Setting initial values of checkboxes (if user has minordefault preference)
* Processing the save action with the minoredit flag (only if has user right and not on a new page)
Comment 2 Krinkle 2012-12-02 05:36:06 UTC 
Change-Id: I9c659845feebb3e9bbf8e13ee67be27c6adb4321
Comment 3 James Forrester 2012-12-03 20:09:25 UTC 
Code now live in master; will get deployed in the next cycle.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links