Update of 
/var/cvs/contributions/CMSContainer_Modules/egemmail/src/webapp/editors/egemmail
In directory 
james.mmbase.org:/tmp/cvs-serv8240/egemmail/src/webapp/editors/egemmail

Modified Files:
        search.jsp 
Log Message:
NIJ-1098 The form submit was triggered twice due to the missing onload event. 
This caused articles to be exported twice.


See also: 
http://cvs.mmbase.org/viewcvs/contributions/CMSContainer_Modules/egemmail/src/webapp/editors/egemmail
See also: http://www.mmbase.org/jira/browse/NIJ-1098


Index: search.jsp
===================================================================
RCS file: 
/var/cvs/contributions/CMSContainer_Modules/egemmail/src/webapp/editors/egemmail/search.jsp,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -b -r1.6 -r1.7
--- search.jsp  18 Jan 2008 12:46:20 -0000      1.6
+++ search.jsp  10 Mar 2008 09:08:52 -0000      1.7
@@ -4,26 +4,29 @@
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" 
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd";>
 <html:html xhtml="true">
 <cmscedit:head title="egemmail.title">
-  <script src="../repository/search.js" type="text/javascript"></script>
+  <script src="../repository/search.js"type="text/javascript"></script>
   <script type="text/javascript">  
     function doForward(to) {
       var elem = document.getElementById("exportForm");
-      if(elem != null) {
+      if (elem != null) {
         elem.forward.value = to;
         elem.submit();
+        return false;
       }
+      return true;
     }
     
     function doChangePage(newPage) {
       var elem = document.getElementById('exportForm');
-      if(elem != null) {
+      if (elem != null) {
         elem.page.value = newPage;
-        doForward('search');
+        return doForward('search');
       }
+      return false;
     }
   </script>
 </cmscedit:head>
-<body onload="refreshChannels()">
+<body>
     <div class="tabs">
         <div class="tab_active">
             <div class="body">
@@ -65,6 +68,7 @@
                 <html:hidden property="author" />
                 <html:hidden property="page" />
                 <html:hidden property="forward" />
+                <html:hidden property="limitToLastWeek" />
                                <mm:list referid="results" 
max="${resultsPerPage}" offset="${offset*resultsPerPage}">
                                    <mm:first>
                                        <egem:paging offset="${offset}" 
resultsPerPage="${resultsPerPage}" 
totalNumberOfResults="${totalNumberOfResults}" />
@@ -121,7 +125,7 @@
                                            </tfoot>
                                        </table>
                         <egem:paging offset="${offset}" 
resultsPerPage="${resultsPerPage}" 
totalNumberOfResults="${totalNumberOfResults}" />                               
    
-                                       <html:submit 
onclick="doForward('export');"><fmt:message key="egemmail.button.export" 
/></html:submit>
+                                       <html:submit onclick="return 
doForward('export');"><fmt:message key="egemmail.button.export" /></html:submit>
                                     </mm:last>
                                 </mm:list>
             </html:form>
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to