These are URI components; fixes comma decoding

Author: sebbASF

www/secretary/workbench/views/parts.js.rb

@@ -587,7 +587,7 @@ def hideMenu(event)
   # burst a PDF into individual pages
   def burst(_event)
     data = {
-      selected: @menu || decodeURI(@selected),
+      selected: @menu || decodeURIComponent(@selected),
       message: window.parent.location.pathname
     }
 
@@ -606,7 +606,7 @@ def burst(_event)
 # join a PDF from individual pages
   def join(_event)
     data = {
-      selected: @menu || decodeURI(@selected),
+      selected: @menu || decodeURIComponent(@selected),
       message: window.parent.location.pathname
     }
 
@@ -650,7 +650,7 @@ def undelete_message(event)
   # delete an attachment
   def delete_attachment(event)
     data = {
-      selected: @menu || decodeURI(@selected),
+      selected: @menu || decodeURIComponent(@selected),
       message: window.parent.location.pathname
     }
 
@@ -678,7 +678,7 @@ def delete_attachment(event)
   # revert to the original
   def revert(_event)
     data = {
-      selected: @menu || decodeURI(@selected),
+      selected: @menu || decodeURIComponent(@selected),
       message: window.parent.location.pathname
     }
 
@@ -702,7 +702,7 @@ def rotate_attachment(event)
     message = window.parent.location.pathname
 
     data = {
-      selected: @menu || decodeURI(@selected),
+      selected: @menu || decodeURIComponent(@selected),
       message: message,
       direction: event.currentTarget.textContent
     }
@@ -726,7 +726,7 @@ def pdfize(_event)
     message = window.parent.location.pathname
 
     data = {
-      selected: @menu || decodeURI(@selected),
+      selected: @menu || decodeURIComponent(@selected),
       message: message
     }
 
@@ -747,7 +747,7 @@ def pdfize(_event)
   # parse pdf and display extracted data
   def pdfparse(_event)
     message = window.parent.location.pathname
-    attachment = @menu || decodeURI(@selected)
+    attachment = @menu || decodeURIComponent(@selected)
     url = message.sub('/workbench/','/icla-parse/') + attachment
     window.parent.frames.content.location.href = url
   end
@@ -906,8 +906,8 @@ def drop(event)
     event.preventDefault()
 
     data = {
-      source: decodeURI(@drag.split('/').pop()),
-      target: decodeURI(href.split('/').pop()),
+      source: decodeURIComponent(@drag.split('/').pop()),
+      target: decodeURIComponent(href.split('/').pop()),
       message: window.parent.location.pathname
     }