pombreda / getpaid

Automatically exported from code.google.com/p/getpaid
0 stars 0 forks source link

javascript repetition + incorrect embedding #266

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
/@@manage-getpaid-content-types
the same javascript appears 3 times ?
and
In
 if (toSel.selectedIndex == -1)
     selectionError();
 else if (toSel.options[0].selected)
     alert("Cannot move further up!");
 else for (var i = 0; i < toSel.length; i++)
   if (toSel.options[i].selected)
     {
     swapFields(toSel.options[i-1], toSel.options[i]);
     copyDataForSubmit(name);
     }
and as said Darryl

embedded javascript should *never* contain the '<' character in an XHTML
document, so this is a bug, one way or the other.

Original issue reported on code.google.com by danielle...@gmail.com on 30 Apr 2009 at 3:12

GoogleCodeExporter commented 9 years ago
the repetition can be avoided using te package zc.resourcelibrary :

The resource library is designed to make the inclusion of JavaScript, CSS, and
other resources easy, cache-friendly, and component-friendly.  For instance, if
two widgets on a page need the same JavaScript library, the library should be
only loaded once, but the widget designers should not have to concern
themselves with the presence of other widgets.

Original comment by danielle...@gmail.com on 3 May 2009 at 2:17