r66282 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r66281‎ | r66282 | r66283 >
Date:19:18, 12 May 2010
Author:dale
Status:deferred
Tags:
Comment:
* Moved apiProxy to separate extension
* updated documentation
Modified paths:
  • /trunk/extensions/AddMediaWizard/AddMediaWizard.php (modified) (history)
  • /trunk/extensions/AddMediaWizard/README (modified) (history)

Diff [purge]

Index: trunk/extensions/AddMediaWizard/AddMediaWizard.php
@@ -50,9 +50,6 @@
5151 // Add the javascript loader for "ClipEdit module"
5252 $wgExtensionJavascriptLoader[] = 'extensions/AddMediaWizard/ClipEdit/loader.js';
5353
54 -// Add the apiProxy ( client ) so that we can upload cross domain to commons
55 -$wgExtensionJavascriptLoader[] = 'extensions/AddMediaWizard/ApiProxy/loader.js';
56 -
5754 class AddMediaWizard {
5855 public static function addJS( $toolbar) {
5956 global $wgOut;
Index: trunk/extensions/AddMediaWizard/README
@@ -1,10 +1,20 @@
2 -# AddMediaWizard is dependent on js2 / mwEmbed, you must include js2Support extension prior to AddMediaWizard.php
 2+# AddMediaWizard is dependent on js2 / mwEmbed, you must include js2Support extension
 3+prior to AddMediaWizard.php
34
45 # To enable add the following to localSettings.php
5 -
66 # JS2 / mwEmbed Support
77 require_once( "$IP/extensions/JS2Support/JS2Support.php" );
8 -
98 # Add Media Wizard
109 require_once( "$IP/extensions/AddMediaWizard/AddMediaWizard.php" );
1110
 11+== Recommended Dependencies ==
 12+AddMediaWiard "should work" without these decencies but you will need to be careful in your
 13+configuration of the extension that you don't request unavailable features
 14+
 15+# To support cross-domain contributions to commons you will also need the apiProxy
 16+javascript module:
 17+require_once( "$IP/extensions/ApiProxy/ApiProxy.php" );
 18+
 19+# To support adding video to pages you will also need the TimedMediaHandler extension:
 20+require_once( "$IP/extensions/TimedMediaHandler/TimedMediaHandler.php" );
 21+

Status & tagging log