Anonymous avatar Anonymous committed 275fd53

[svn] Fixed parallel_effects to call effects properly, reported by Tiago Alves.

Comments (0)

Files changed (1)

webhelpers/rails/scriptaculous.py

             visual_effect('fade', 'dom_id'),
         )
     """
-    return "new Effect.Parallel([%s], %s)" % (effects.join(''), options_for_javascript(js_options))
+    str_effects = [e[:-1] for e in effects] # Remove trailing ';'
+    return "new Effect.Parallel([%s], %s)" % (','.join(str_effects), options_for_javascript(js_options))
 
 def sortable_element(element_id, **options):
     """
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.