Discussion MediaWiki:Gadget-CommonEdit.js

Remove redundant check

Hi!

I think we can change "forcePreview" code from

  if (wgUserName != null || wgAction != "edit") return;

to

  if (wgUserName != null) return;

because MediaWiki:Common.js/edit.js is only loaded when edit a page. (Or this would cause a problem with "|| wgAction == 'submit'" at MediaWiki:Common.js?) Heldergeovane (d) 20 juillet 2009 à 14:11 (CEST)

I'm not sure to understand: MediaWiki:Common.js/edit.js must be loaded both when the action is "edit" and "submit" (for instance the special caracters are loaded whenever during editing); but forcePreview must be loaded only the first time an editpage is loaded, else anonymous visitors couldn't never edit, isn't it? ~ Seb35 [^_^] 5 octobre 2009 à 19:40 (CEST)

Hello!

I was wondering if is it possible to check the value of wgAction only one time. Currently this is checked twice, at MediaWiki:Common.js:

if( wgAction == 'edit' || wgAction == 'submit' ) {
  importScript( 'MediaWiki:Common.js/edit.js' );
}

and at MediaWiki:Common.js/edit.js:

  if (wgUserName != null || wgAction != "edit") return;

But looking at the code now it seems not possible to do this simplification... What do you think? Heldergeovane (d) 17 octobre 2009 à 21:33 (CEST)