saper has uploaded a new change for review.

Change subject: (bug 37419) Rewriting of Index pages edit form in PHP
......................................................................

(bug 37419) Rewriting of Index pages edit form in PHP

Rewriting of the index form in PHP in order to improve performance (the edit 
toolbars were loaded and showed and then Prp created the form) and allow easy 
editing of index pages
Creation of json configuration message proofreadpage_index_data_config that 
replace proofreadpage_index_attributes and proofreadpage_js_attributes.
Wiki can keep using the two old messages. The description of the syntax is done 
in ProofreadIndexPage::getDataConfig()
This creation allow to add new features in the form:
- list of possible values for a field (show as a select html element)
- help messages
- type for each data
- default values
There is no validation in this form in order to support existant data.
The names and id of field has change: a prefix "wpprpindex-" is added in order 
to avoid any id conflict

Some generic classes where added:
- ProofreadIndexPage that offer methods in order to get index page content.
- ProofreadIndexEntry that manage an entry in the index page and is 
configuration.
Use of these two classes inside of existing parts of the ProofreeadPage 
extension will be done in an other commit.

History:
 gerrit change https://gerrit.wikimedia.org/r/#/c/10741/
 Old-Id: Iaade8c7ee83035387be16e24d35e29c45a5717b2
Patch set 1-8: creation of a new form generator in Javascript
Patch set 9: Rebase + change name of config message and various small 
improvements
Patch set 10: Writing of PHP form generator.
Patch set 11: Fix bug: The PHP form didn't works in preview mode + small 
improvements
Patch set 12: Fix some bugs in cleanInputtedContent (see inline comments of 
Patch set 11 in EditProofreadIndexPage.php)
Patch set 13: Rebase + some UI improvements

Change-Id: I942304918b780ef97f516aef8cb9f962a44de627
---
M ProofreadPage.body.php
M ProofreadPage.i18n.php
M ProofreadPage.php
A includes/index/EditProofreadIndexPage.php
A includes/index/ProofreadIndexEntry.php
A includes/index/ProofreadIndexPage.php
A modules/ext.proofreadpage.index/ext.proofreadpage.index.css
M modules/ext.proofreadpage.index/ext.proofreadpage.index.js
8 files changed, 605 insertions(+), 145 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ProofreadPage 
refs/changes/99/27199/1
--
To view, visit https://gerrit.wikimedia.org/r/27199
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I942304918b780ef97f516aef8cb9f962a44de627
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ProofreadPage
Gerrit-Branch: master
Gerrit-Owner: saper <[email protected]>
Gerrit-Reviewer: Tpt <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to