Author: snoopdave
Date: Fri Aug 10 09:27:25 2007
New Revision: 564654

URL: http://svn.apache.org/viewvc?view=rev&rev=564654
Log:
Fixes "Drafts erroneously published if user hits enter while in title or tags 
field"
http://opensource.atlassian.com/projects/roller/browse/ROL-1525

There may be a more sophisticated fix that allows us to maintain the old button 
order, but the easiest fix here is to put the draft button first in the line-up.

I tried to disable the "enter" button entirely via onsubmit="return false" and 
postToWeblog() saveAsDraft() and submitForReview() methods, but the Struts2 
<s:form> element changes that to "return false; return true", so I backed off 
to this more simple approach.

Modified:
    roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryAdd.jsp
    roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryEdit.jsp

Modified: roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryAdd.jsp
URL: 
http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryAdd.jsp?view=diff&rev=564654&r1=564653&r2=564654
==============================================================================
--- roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryAdd.jsp (original)
+++ roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryAdd.jsp Fri Aug 10 
09:27:25 2007
@@ -181,13 +181,13 @@
     
     <br>
     <div class="control">
+        <s:submit key="weblogEdit.save" 
onclick="$('entry_bean_status').value='DRAFT';" />
         <s:if test="userAnAuthor">
             <s:submit key="weblogEdit.post" 
onclick="$('entry_bean_status').value='PUBLISHED';"/>
         </s:if>
         <s:else>
             <s:submit key="weblogEdit.submitForReview" 
onclick="$('entry_bean_status').value='PENDING';"/>
         </s:else>
-        <s:submit key="weblogEdit.save" 
onclick="$('entry_bean_status').value='DRAFT';" />
     </div>
     
 </s:form>

Modified: roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryEdit.jsp
URL: 
http://svn.apache.org/viewvc/roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryEdit.jsp?view=diff&rev=564654&r1=564653&r2=564654
==============================================================================
--- roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryEdit.jsp (original)
+++ roller/trunk/apps/weblogger/web/WEB-INF/jsps/editor/EntryEdit.jsp Fri Aug 
10 09:27:25 2007
@@ -238,14 +238,14 @@
     
     <br>
     <div class="control">
+        <s:submit key="weblogEdit.save" 
onclick="$('entry_bean_status').value='DRAFT';" />
+
         <s:if test="userAnAuthor">
             <s:submit key="weblogEdit.post" 
onclick="$('entry_bean_status').value='PUBLISHED';"/>
         </s:if>
         <s:else>
             <s:submit key="weblogEdit.submitForReview" 
onclick="$('entry_bean_status').value='PENDING';"/>
-        </s:else>
-        
-        <s:submit key="weblogEdit.save" 
onclick="$('entry_bean_status').value='DRAFT';" />
+        </s:else>        
         
         <s:url id="removeUrl" action="entryRemove">
             <s:param name="weblog" value="actionWeblog.handle" />


Reply via email to