[Patch] Option for using JQuery from a CDN

[psuter <petsuter@…>]

Add the option [trac] jquery_location:

trac/web/chrome.py

@@ -362 +362 @@
         will not be made available this way and additional rewrite 
         rules will be needed in the web server.""")
 
+    jquery_location = Option('trac', 'jquery_location', '',
+        """CDN URL for JQuery.
+        
+        It can be left empty, and the JQuery bundled with Trac will be used.
+        
+        Alternatively you can deliver JQuery from a CDN by setting it to a URL 
+        like http://code.jquery.com/jquery-1.5.1.min.js, 
+        http://ajax.aspnetcdn.com/ajax/jQuery/jquery-1.5.1.min.js or
+        https://ajax.googleapis.com/ajax/libs/jquery/1.5.1/jquery.min.js.""")
+
     metanav_order = ListOption('trac', 'metanav',
                                'login, logout, prefs, help, about', doc=
         """Order of the items to display in the `metanav` navigation bar,
@@ -605 +615 @@
         add_link(req, 'search', req.href.search())
         add_link(req, 'help', req.href.wiki('TracGuide'))
         add_stylesheet(req, 'common/css/trac.css')
-        add_script(req, 'common/js/jquery.js')
+        add_script(req, self.jquery_location or 'common/js/jquery.js')
         # Only activate noConflict mode if requested to by the handler
         if handler is not None and \
            getattr(handler.__class__, 'jquery_noconflict', False):

as discussed by comment:16:ticket:9643 and comment:17:ticket:9643.

[Remy Blank]

Looks good. We should make sure the docstring always references the same version of jQuery as bundled with Trac.

[Remy Blank]

Patch applied in [10733] and [10734]. Thanks!

[psuter <petsuter@…>]

I fell in the exact same trap with the formatting. Nice to know I'm in good company. :)