Author: kcardineaud Date: 2011-08-09 15:56:44 +0200 (Tue, 09 Aug 2011) New Revision: 209 Url: http://nuiton.org/repositories/revision/scmwebeditor/209 Log: Add a confirmation message if the rst is not valid when we save the file Modified: trunk/src/main/java/org/nuiton/scmwebeditor/actions/ScmWebEditorCommitAction.java trunk/src/main/webapp/WEB-INF/content/modificationViewer.jsp trunk/src/main/webapp/WEB-INF/content/save.jsp Modified: trunk/src/main/java/org/nuiton/scmwebeditor/actions/ScmWebEditorCommitAction.java =================================================================== --- trunk/src/main/java/org/nuiton/scmwebeditor/actions/ScmWebEditorCommitAction.java 2011-08-09 09:22:24 UTC (rev 208) +++ trunk/src/main/java/org/nuiton/scmwebeditor/actions/ScmWebEditorCommitAction.java 2011-08-09 13:56:44 UTC (rev 209) @@ -42,8 +42,13 @@ protected String lastText; protected String fileType; protected String projectUrl; + protected boolean force; + + + + protected HttpServletRequest request; public String getCommitMessage() { @@ -119,7 +124,11 @@ this.projectUrl = projectUrl; } + public void setForce(boolean force) { + this.force = force; + } + protected boolean testRstParsing(String newText) { try { @@ -142,14 +151,15 @@ public String execute() { - if(fileType.equals("rst")) { - if(!testRstParsing(newText)) { - return "error"; - } - } + if(!force) { + if(fileType.equals("rst")) { + if(!testRstParsing(newText)) { + return "errorRst"; + } + } + } - SvnConnection svnConn = null; @@ -172,8 +182,9 @@ repositoryUUID=address; } + log.info("username : |"+login+"|"); - if(login==null && password==null) { + if( ( login==null ) && ( password==null ) ) { if(getScmSession().getUsername(repositoryUUID)!=null && getScmSession().getPassword(repositoryUUID)!=null) { //On recupère les identifiants en session login = getScmSession().getUsername(repositoryUUID); Modified: trunk/src/main/webapp/WEB-INF/content/modificationViewer.jsp =================================================================== --- trunk/src/main/webapp/WEB-INF/content/modificationViewer.jsp 2011-08-09 09:22:24 UTC (rev 208) +++ trunk/src/main/webapp/WEB-INF/content/modificationViewer.jsp 2011-08-09 13:56:44 UTC (rev 209) @@ -65,7 +65,7 @@ </center> - <form method="POST" action=commit.action id="editForm"> + <form method="POST" action="commit.action?force=true" id="editForm"> <% if (request.getAttribute("format")!=null && request.getAttribute("format").equals("rst")){ %> Modified: trunk/src/main/webapp/WEB-INF/content/save.jsp =================================================================== --- trunk/src/main/webapp/WEB-INF/content/save.jsp 2011-08-09 09:22:24 UTC (rev 208) +++ trunk/src/main/webapp/WEB-INF/content/save.jsp 2011-08-09 13:56:44 UTC (rev 209) @@ -9,6 +9,24 @@ <s:elseif test="result=='error'" > <h4> Can't save modification </h4> </s:elseif> + <s:elseif test="result=='errorRst'" > + + <h4> The rst is not valid, do you still want to continue ? </h4> + <s:url id="ajaxForceSave" action="save" /> + <input type="hidden" name="force" value="true"/> + <sj:submit onclick="loadChange()" + id="ajaxForceSaveButton" + formIds="editForm" + targets="htmlcontentCommit" + href="%{ajaxForceSave}" + indicator="indicator" + button="true" + buttonIcon="ui-icon-refresh" + value="YES" + > + </sj:submit> + + </s:elseif> <s:elseif test="result=='errorPath'" > <h4> Can't save modification, RST is not valid</h4> </s:elseif>