February 2011 - Coser-commits - mailman.cloud.codelutin.com

r632 - in trunk: . coser-business/src/main/java/fr/ifremer/coser/services coser-business/src/test/java/fr/ifremer/coser/services coser-business/src/test/resources coser-ui/src/main/java/fr/ifremer/coser/ui/result coser-web/src/main/java/fr/ifremer/coser/web/actions/map coser-web/src/main/java/fr/ifremer/coser/web/actions/source coser-web/src/main/webapp/WEB-INF/content/source coser-web/src/main/webapp/js coser-web/src/main/webapp/styles
by chatellier＠users.labs.libre-entreprise.org 01 Feb '11

01 Feb '11

Author: chatellier Date: 2011-02-01 13:19:15 +0000 (Tue, 01 Feb 2011) New Revision: 632 Log: Add facadeid. Manage quelity advertisment in form submit. Added: trunk/coser-web/src/main/webapp/WEB-INF/content/source/source-data-input.jsp Modified: trunk/changelog.txt trunk/coser-business/src/main/java/fr/ifremer/coser/services/WebService.java trunk/coser-business/src/test/java/fr/ifremer/coser/services/WebServiceTest.java trunk/coser-business/src/test/resources/webzones.csv trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/result/ZoneComboBoxModel.java trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/result/ZoneComboBoxRenderer.java trunk/coser-web/src/main/java/fr/ifremer/coser/web/actions/map/MapAction.java trunk/coser-web/src/main/java/fr/ifremer/coser/web/actions/source/SourceDataAction.java trunk/coser-web/src/main/webapp/WEB-INF/content/source/source-quality.jsp trunk/coser-web/src/main/webapp/js/coser.js trunk/coser-web/src/main/webapp/styles/coser.css Modified: trunk/changelog.txt =================================================================== --- trunk/changelog.txt 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/changelog.txt 2011-02-01 13:19:15 UTC (rev 632) @@ -4,6 +4,7 @@ 1.0.0-beta3 ----------- + * Improve quality advertisement checkbox with form submit * Add top anchor on survey description page * Fix pdf generation containing non escaped html entities * List projects names sorted alphabetically Modified: trunk/coser-business/src/main/java/fr/ifremer/coser/services/WebService.java =================================================================== --- trunk/coser-business/src/main/java/fr/ifremer/coser/services/WebService.java 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-business/src/main/java/fr/ifremer/coser/services/WebService.java 2011-02-01 13:19:15 UTC (rev 632) @@ -42,6 +42,7 @@ import java.util.Date; import java.util.HashMap; import java.util.Iterator; +import java.util.LinkedHashMap; import java.util.List; import java.util.Locale; import java.util.Map; @@ -221,13 +222,13 @@ public String getZoneFullName(String zoneId) throws CoserBusinessException { DataStorage localZoneMap = getZonesMap(); String resultName = null; - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" int zoneIndex = localZoneMap.indexOf(zoneId); if (zoneIndex != -1) { - resultName = localZoneMap.get(zoneIndex)[1]; - resultName += " - " + localZoneMap.get(zoneIndex)[2]; + resultName = localZoneMap.get(zoneIndex)[2]; resultName += " - " + localZoneMap.get(zoneIndex)[3]; resultName += " - " + localZoneMap.get(zoneIndex)[4]; + resultName += " - " + localZoneMap.get(zoneIndex)[5]; } return resultName; } @@ -1017,14 +1018,14 @@ * @throws CoserBusinessException */ public Map<String, String> getFacades() throws CoserBusinessException { - Map<String, String> facades = new HashMap<String, String>(); - // "id";"facade";"zone";"periode";"serie";"comment";"map" + Map<String, String> facades = new LinkedHashMap<String, String>(); + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" DataStorage zonesMap = getZonesMap(); Iterator<String[]> itZone = zonesMap.iterator(true); while (itZone.hasNext()) { String[] tuple = itZone.next(); // on a pas d'id pour les facades - facades.put(tuple[1], tuple[1]); + facades.put(tuple[1], tuple[2]); } return facades; } @@ -1045,7 +1046,7 @@ Collection<String> subZones = new ArrayList<String>(); Iterator<String[]> itZone = getZonesMap().iterator(true); while (itZone.hasNext()) { - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" String[] tuple = itZone.next(); if (tuple[1].equals(facade)) { subZones.add(tuple[0]); @@ -1080,10 +1081,10 @@ if (subZones.contains(resultZoneId)) { String zoneid = resultZoneId; // get zone name - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" int zoneIndex = getZonesMap().indexOf(zoneid); String[] zoneData = getZonesMap().get(zoneIndex); - String zoneName = zoneData[2] + " - " + zoneData[3] + " - " + zoneData[4]; + String zoneName = zoneData[3] + " - " + zoneData[4] + " - " + zoneData[5]; zonesForFacade.put(zoneid, zoneName); } @@ -1115,11 +1116,11 @@ // get subzone for main zone Iterator<String[]> itZone = getZonesMap().iterator(true); while (itZone.hasNext()) { - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" String[] tuple = itZone.next(); if (tuple[1].equals(facade)) { - if (!surveyForFacade.containsKey(tuple[4])) { - surveyForFacade.put(tuple[4], tuple[4]); + if (!surveyForFacade.containsKey(tuple[5])) { + surveyForFacade.put(tuple[5], tuple[5]); } } } @@ -1138,9 +1139,9 @@ Iterator<String[]> itZone = getZonesMap().iterator(true); while (itZone.hasNext()) { - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" String[] tuple = itZone.next(); - result.put(tuple[0], tuple[6]); + result.put(tuple[0], tuple[7]); } return result; @@ -1157,9 +1158,9 @@ Iterator<String[]> itZone = getZonesMap().iterator(true); while (itZone.hasNext()) { - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" String[] tuple = itZone.next(); - result.put(tuple[0], tuple[5]); + result.put(tuple[0], tuple[6]); } return result; @@ -1265,10 +1266,10 @@ boolean zoneRSufiResult = false; Iterator<String[]> itZone = getZonesMap().iterator(true); while (itZone.hasNext()) { - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" String[] tuple = itZone.next(); - if (tuple[4].equals(survey) && rsufiResult.getZone().equals(tuple[0])) { + if (tuple[5].equals(survey) && rsufiResult.getZone().equals(tuple[0])) { zoneRSufiResult = true; } } @@ -1713,10 +1714,10 @@ boolean zoneRSufiResult = false; Iterator<String[]> itZone = getZonesMap().iterator(true); while (itZone.hasNext()) { - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" String[] tuple = itZone.next(); - if (tuple[4].equals(survey) && rsufiResult.getZone().equals(tuple[0])) { + if (tuple[5].equals(survey) && rsufiResult.getZone().equals(tuple[0])) { zoneRSufiResult = true; } } Modified: trunk/coser-business/src/test/java/fr/ifremer/coser/services/WebServiceTest.java =================================================================== --- trunk/coser-business/src/test/java/fr/ifremer/coser/services/WebServiceTest.java 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-business/src/test/java/fr/ifremer/coser/services/WebServiceTest.java 2011-02-01 13:19:15 UTC (rev 632) @@ -99,7 +99,7 @@ @Test public void testGetZones() throws CoserBusinessException { DataStorage zones = webService.getZonesMap(); - Assert.assertEquals(17, zones.size()); + Assert.assertEquals(21, zones.size()); } /** Modified: trunk/coser-business/src/test/resources/webzones.csv =================================================================== --- trunk/coser-business/src/test/resources/webzones.csv 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-business/src/test/resources/webzones.csv 2011-02-01 13:19:15 UTC (rev 632) @@ -1,19 +1,21 @@ -"id";"zone";"subzone";"periode";"serie";"comment";"map" -"ciem4b4c";"Mer du Nord";"CIEM 4b - 4c";"1983";"IBTS";"D'après données internationales extraites de la base Datras du CIEM";"C_Ciem-4b-4c.jpg" -"ciem4c";"Mer du Nord";"CIEM 4c";"1983";"IBTS";"D'après données internationales extraites de la base Datras du CIEM";"C_Ciem-4c.jpg" -"ciem4ce7d";"Mer du Nord";"CIEM 4c - Est 7d";"2007";"IBTS";"D'après données internationales extraites de la base Datras du CIEM";"C_Ciem-4c-E7d.jpg" -"ciem4c4e7d";"Mer du Nord";"CIEM 4c4 - Est 7d";"2007";"IBTS";"- Détroit du Pas-de-Calais\n- D'après données internationales extraites de la base Datras du CIEM";"C_Ciem-4c4-E7d.jpg" -"cieme7d";"Manche orientale";"Est de 1° 15' O";"1988 ";"CGFS";;"C_Ciem-E7d.jpg" -"ciem7d";"Manche orientale";"Ensemble Manche orientale";"1990";"CGFS";"Correspond à l'ensemble de la zone VIId du CIEM";"C_Ciem-7d.jpg" -"noursom";"Manche orientale";"Baie de Somme";"1994";"Noursom";;"C_Noursom.jpg" -"noursein";"Manche orientale";"Baie de Seine";"1995 - 2010";"Noursein";"Série arrêtée en 2010";"C_Noursein.jpg" -"crustaflam";"Manche occidentale";"Abords du cap de Flamanville";"1986";"Crustaflam";;"C_Crustaflam.jpg" -"gdgmc";"Atlantique";"Golfe de Gascogne + mer Celtique";"1997";"Evhoe";"Ensemble de la couverture Evhoe";"C_GdG-MC.jpg" -"mcech";"Atlantique";"Mer Celtique (strates d'échantillonnage)";"1997";"Evhoe";;"C_MC-Ech.jpg" -"mcciem7";"Atlantique";"Mer Celtique (zone VII du CIEM)";"1997";"Evhoe";"La limite sud de la zone VII du CIEM (48 °N) constitue aussi une limite de régions de la DCSMM";"C_MC-Ciem-7.jpg" -"gdgech";"Atlantique";"Golfe de Gascogne (strates d'échantillonnage)";"1987";"Evhoe";;"C_GdG-Ech.jpg" -"gdgciem7";"Atlantique";"Golfe de Gascogne (zone VIII du CIEM)";"1987";"Evhoe";"La limite nord de la zone VIII du CIEM (48 °N) constitue aussi une limite de régions de la DCSMM";"C_GdG-Ciem-7.jpg" -"gdl";"Méditerranée";"Golfe du Lion";"1994";"Medits";;"C_GdL.jpg" -"ecorse";"Méditerranée";"Est-Corse";"1994";"Medits";"1997 exclue (couverture incomplète)";"C_E-Corse.jpg" - - +"id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" +"northibts";"merdunord";"Mer du Nord";"Toutes zones IBTS au sud de 57° 30' N";"1983 >";"IBTS";"Application prioritaire : cartes de distribution des espèces";"C_Ciem-4b-4c-E7d.png" +"northibts2";"merdunord";"Mer du Nord";"Toutes zones IBTS au sud de 57° 30' N";"2007 >";"IBTS";"Application prioritaire : cartes de distribution des espèces";"C_Ciem-4b-4c.png" +"ciem4b4c";"merdunord";"Mer du Nord";"CIEM 4b - 4c";"1983 >";"IBTS";"D'après données internationales extraites de la base Datras du CIEM";"C_Ciem-4b-4c.png" +"ciem4c";"merdunord";"Mer du Nord";"CIEM 4c";"1983 >";"IBTS";"D'après données internationales extraites de la base Datras du CIEM";"C_Ciem-4c.png" +"ciem4ce7d";"merdunord";"Mer du Nord";"CIEM 4c - Est 7d";"2007 >";"IBTS";"D'après données internationales extraites de la base Datras du CIEM";"C_Ciem-4c-E7d.png" +"ciem4c4e7d";"merdunord";"Mer du Nord";"CIEM 4c4 - Est 7d";"2007 >";"IBTS";"- Détroit du Pas-de-Calais\n- D'après données internationales extraites de la base Datras du CIEM";"C_Ciem-4c4-E7d.png" +"cieme7d";"mancheorientale";"Manche orientale";"Est de 1° 15' O";"1988 >";"CGFS";;"C_Ciem-E7d.png" +"ciem7d";"mancheorientale";"Manche orientale";"Ensemble Manche orientale";"1990 >";"CGFS";"Correspond à l'ensemble de la zone VIId du CIEM";"C_Ciem-7d.png" +"noursom";"mancheorientale";"Manche orientale";"Baie de Somme";"1994 >";"Noursom";;"C_Noursom.png" +"noursein";"mancheorientale";"Manche orientale";"Baie de Seine";"1995 - 2010";"Noursein";"Série arrêtée en 2010";"C_Noursein.png" +"crustaflam";"mancheoccidentale";"Manche occidentale";"Abords du cap de Flamanville";"1986 >";"Crustaflam";;"C_Crustaflam.png" +"gdgmc";"atlantique";"Atlantique";"Golfe de Gascogne + mer Celtique";"1997 >";"Evhoe";"Ensemble de la couverture Evhoe";"C_GdG-MC.png" +"mcech";"atlantique";"Atlantique";"Mer Celtique (strates d'échantillonnage)";"1997 >";"Evhoe";;"C_MC-Ech.png" +"mcciem7";"atlantique";"Atlantique";"Mer Celtique (zone VII du CIEM)";"1997 >";"Evhoe";"La limite sud de la zone VII du CIEM (48 °N) constitue aussi une limite de régions de la DCSMM";"C_MC-Ciem-7.png" +"gdgech";"atlantique";"Atlantique";"Golfe de Gascogne (strates d'échantillonnage)";"1987 >";"Evhoe";;"C_GdG-Ech.png" +"gdgech2";"atlantique";"Atlantique";"Golfe de Gascogne (strates d'échantillonnage)";"1992 >";"Evhoe";"Plus d'espèces mesurées à partir de 1992";"C_GdG-Ech.png" +"gdgciem7";"atlantique";"Atlantique";"Golfe de Gascogne (zone VIII du CIEM)";"1987 >";"Evhoe";"La limite nord de la zone VIII du CIEM (48 °N) constitue aussi une limite de régions de la DCSMM";"C_GdG-Ciem-7.png" +"gdgciem72";"atlantique";"Atlantique";"Golfe de Gascogne (zone VIII du CIEM)";"1992 >";"Evhoe";"- La limite nord de la zone VIII du CIEM (48 °N) constitue aussi une limite de régions de la DCSMM\n- Plus d'espèces mesurées à partir de 1992";"C_GdG-Ciem-7.png" +"gdl";"mediteranee";"Méditerranée";"Golfe du Lion";"1994 >";"Medits";;"C_GdL.png" +"ecorse";"mediteranee";"Méditerranée";"Est-Corse";"1994 >";"Medits";"1997 exclue (couverture incomplète)";"C_E-Corse.png" Modified: trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/result/ZoneComboBoxModel.java =================================================================== --- trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/result/ZoneComboBoxModel.java 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/result/ZoneComboBoxModel.java 2011-02-01 13:19:15 UTC (rev 632) @@ -64,7 +64,7 @@ @Override public Object getElementAt(int index) { - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" // +1 for header return zonesMap.get(index + 1)[0]; } Modified: trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/result/ZoneComboBoxRenderer.java =================================================================== --- trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/result/ZoneComboBoxRenderer.java 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-ui/src/main/java/fr/ifremer/coser/ui/result/ZoneComboBoxRenderer.java 2011-02-01 13:19:15 UTC (rev 632) @@ -49,14 +49,14 @@ String zoneId = (String)value; ZoneComboBoxModel model = (ZoneComboBoxModel)list.getModel(); - // "id";"facade";"zone";"periode";"serie";"comment";"map" + // "id";"facadeid";"facade";"zone";"periode";"serie";"comment";"map" String[] zoneData = model.getZone(zoneId); String zoneName = null; if (zoneData != null) { // le surveyName ne sert à rien // mais est la a titre d'information par rapport // à l'affichage cote web - zoneName = zoneData[1] + " - " + zoneData[2] + " - " + zoneData[3] + " - " + zoneData[4]; + zoneName = zoneData[2] + " - " + zoneData[3] + " - " + zoneData[4] + " - " + zoneData[5]; } else { zoneName = zoneId; Modified: trunk/coser-web/src/main/java/fr/ifremer/coser/web/actions/map/MapAction.java =================================================================== --- trunk/coser-web/src/main/java/fr/ifremer/coser/web/actions/map/MapAction.java 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-web/src/main/java/fr/ifremer/coser/web/actions/map/MapAction.java 2011-02-01 13:19:15 UTC (rev 632) @@ -67,11 +67,22 @@ public String getSpecies() { return species; } - + public void setSpecies(String species) { this.species = species; } - + + public String getFacadeDisplayName() { + WebService webService = ServiceFactory.getWebService(); + String displayName = null; + try { + displayName = webService.getFacades().get(facade); + } catch (CoserBusinessException ex) { + throw new CoserWebException("Can't get species display name", ex); + } + return displayName; + } + public String getSpeciesDisplayName() { WebService webService = ServiceFactory.getWebService(); String displayName = null; Modified: trunk/coser-web/src/main/java/fr/ifremer/coser/web/actions/source/SourceDataAction.java =================================================================== --- trunk/coser-web/src/main/java/fr/ifremer/coser/web/actions/source/SourceDataAction.java 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-web/src/main/java/fr/ifremer/coser/web/actions/source/SourceDataAction.java 2011-02-01 13:19:15 UTC (rev 632) @@ -55,6 +55,8 @@ protected String zone; + protected boolean accepted; + public String getZone() { return zone; } @@ -63,6 +65,19 @@ this.zone = zone; } + public void setAccepted(boolean accepted) { + this.accepted = accepted; + } + + @Override + public void validate() { + + if (!accepted) { + addFieldError("accepted", getText("message.quality.notaccepted")); + } + + } + @Action(results= {@Result(type="stream", params={"contentType", "application/zip", "inputName", "inputStream", "contentDisposition", "attachment; filename=\"${filename}\""})}) public String execute() { return SUCCESS; Added: trunk/coser-web/src/main/webapp/WEB-INF/content/source/source-data-input.jsp =================================================================== --- trunk/coser-web/src/main/webapp/WEB-INF/content/source/source-data-input.jsp (rev 0) +++ trunk/coser-web/src/main/webapp/WEB-INF/content/source/source-data-input.jsp 2011-02-01 13:19:15 UTC (rev 632) @@ -0,0 +1,55 @@ + +<%@ page language="java" contentType="text/html; charset=utf-8" pageEncoding="utf-8"%> +<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> +<%@taglib uri="/struts-tags" prefix="s" %> +<html xmlns="http://www.w3.org/1999/xhtml"> + <head> + <title><s:text name="message.source.title" /></title> + </head> + <body> + + <h2><s:text name="message.source.title" /></h2> + + <h3><s:text name="message.quality.title" /></h3> + + <p><s:text name="message.quality.paragraph1" /></p> + <ul> + <li><s:text name="message.quality.paragraph2" /></li> + <li><s:text name="message.quality.paragraph3" /></li> + <li><s:text name="message.quality.paragraph4" /></li> + </ul> + <p> + <s:text name="message.quality.paragraph5"> + <s:param><a href="mailto:<s:property value='adminEmail' />"><s:property value='adminEmail' /></a></s:param> + </s:text> + </p> + + <s:form action="source-data"> + <s:hidden name="zone" property="zone"/> + <s:checkbox name="accepted" fieldValue="true" label="%{getText('message.quality.acceptance')}"/> + <s:submit value="%{getText('message.source.download')}"></s:submit> + </s:form> + </body> +</html> \ No newline at end of file Modified: trunk/coser-web/src/main/webapp/WEB-INF/content/source/source-quality.jsp =================================================================== --- trunk/coser-web/src/main/webapp/WEB-INF/content/source/source-quality.jsp 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-web/src/main/webapp/WEB-INF/content/source/source-quality.jsp 2011-02-01 13:19:15 UTC (rev 632) @@ -46,14 +46,10 @@ </s:text> </p> - <p> - <input type="checkbox" id="coserResultCondition" /> - <s:text name="message.quality.acceptance" /> - </p> - - <s:a action="source-data" onclick="return coserCheckConditionAcceptance('coserResultCondition', '%{getText('message.quality.notaccepted')}')"> - <s:param name="zone" value="%{zone}" /> - <s:text name="message.source.download"></s:text> - </s:a>. + <s:form action="source-data"> + <s:hidden name="zone" property="zone"/> + <s:checkbox name="accepted" fieldValue="true" label="%{getText('message.quality.acceptance')}"/> + <s:submit value="%{getText('message.source.download')}"></s:submit> + </s:form> </body> </html> \ No newline at end of file Modified: trunk/coser-web/src/main/webapp/js/coser.js =================================================================== --- trunk/coser-web/src/main/webapp/js/coser.js 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-web/src/main/webapp/js/coser.js 2011-02-01 13:19:15 UTC (rev 632) @@ -5,13 +5,3 @@ // affiche la carte selectionnee $('#' + prefix + $(select).val()).show(); } - -// affiche un message si l'utilsateur n'a pas validé les conditions -function coserCheckConditionAcceptance(checkboxid, message) { - var result = true; - if (!$('#' + checkboxid).attr('checked')) { - window.alert(message); - result = false; - } - return result; -} \ No newline at end of file Modified: trunk/coser-web/src/main/webapp/styles/coser.css =================================================================== --- trunk/coser-web/src/main/webapp/styles/coser.css 2011-02-01 10:18:04 UTC (rev 631) +++ trunk/coser-web/src/main/webapp/styles/coser.css 2011-02-01 13:19:15 UTC (rev 632) @@ -693,3 +693,7 @@ background-color: #fff; border: 1px solid #ccc; }*/ + +.errorMessage { + color:red; +}

1 0

r631 - in trunk: . coser-web/src/main/resources/fr/ifremer/coser/web coser-web/src/main/webapp/WEB-INF/content
by chatellier＠users.labs.libre-entreprise.org 01 Feb '11

01 Feb '11

Author: chatellier Date: 2011-02-01 10:18:04 +0000 (Tue, 01 Feb 2011) New Revision: 631 Log: Add top anchor Modified: trunk/changelog.txt trunk/coser-web/src/main/resources/fr/ifremer/coser/web/package.properties trunk/coser-web/src/main/resources/fr/ifremer/coser/web/package_fr.properties trunk/coser-web/src/main/webapp/WEB-INF/content/survey.jsp Modified: trunk/changelog.txt =================================================================== --- trunk/changelog.txt 2011-02-01 09:37:07 UTC (rev 630) +++ trunk/changelog.txt 2011-02-01 10:18:04 UTC (rev 631) @@ -4,6 +4,7 @@ 1.0.0-beta3 ----------- + * Add top anchor on survey description page * Fix pdf generation containing non escaped html entities * List projects names sorted alphabetically * Fix website texts Modified: trunk/coser-web/src/main/resources/fr/ifremer/coser/web/package.properties =================================================================== --- trunk/coser-web/src/main/resources/fr/ifremer/coser/web/package.properties 2011-02-01 09:37:07 UTC (rev 630) +++ trunk/coser-web/src/main/resources/fr/ifremer/coser/web/package.properties 2011-02-01 10:18:04 UTC (rev 631) @@ -45,6 +45,7 @@ message.index.qualitymessage=Avertissement qualit\u00E9 message.index.surveytitle=Description des campagnes message.index.surveyparagraph=Des manuels des protocoles d\u00E9crivent les modalit\u00E9s techniques de r\u00E9alisation de chaque s\u00E9rie de campagnes. +message.common.anchortop=Top message.common.selectpop=Select a population message.common.selectzone=Select a zone message.common.selectsurvey=Select a survey Modified: trunk/coser-web/src/main/resources/fr/ifremer/coser/web/package_fr.properties =================================================================== --- trunk/coser-web/src/main/resources/fr/ifremer/coser/web/package_fr.properties 2011-02-01 09:37:07 UTC (rev 630) +++ trunk/coser-web/src/main/resources/fr/ifremer/coser/web/package_fr.properties 2011-02-01 10:18:04 UTC (rev 631) @@ -45,6 +45,7 @@ message.index.qualitymessage=Avertissement qualit\u00E9 message.index.surveytitle=Description des campagnes message.index.surveyparagraph=Des manuels des protocoles d\u00E9crivent les modalit\u00E9s techniques de r\u00E9alisation de chaque s\u00E9rie de campagnes. +message.common.anchortop=Haut message.common.selectpop=S\u00E9lectionnez une population message.common.selectzone=S\u00E9lectionnez une zone message.common.selectsurvey=S\u00E9lectionnez une campagne Modified: trunk/coser-web/src/main/webapp/WEB-INF/content/survey.jsp =================================================================== --- trunk/coser-web/src/main/webapp/WEB-INF/content/survey.jsp 2011-02-01 09:37:07 UTC (rev 630) +++ trunk/coser-web/src/main/webapp/WEB-INF/content/survey.jsp 2011-02-01 10:18:04 UTC (rev 631) @@ -32,7 +32,7 @@ $(document).ready(function() { // override some default options $('div.expandable').expander({ - slicePoint: 1560, // default is 100 + slicePoint: 1565, // default is 100 expandText: '<s:text name="message.common.jsreadmore" />', // default is 'read more...' collapseTimer: 0, // re-collapses after 5 seconds; default is 0, so no re-collapsing userCollapseText: '[^]' // default is '[collapse expanded text]' @@ -42,7 +42,7 @@ </script> </head> <body> - + <a name="top"></a> <h2> <s:text name="message.survey.maintitle" /> </h2> @@ -111,7 +111,10 @@ <a name="merdunord"></a> <h3><s:text name="message.survey.merdunord" /></h3> <a name="sudmerdunord"></a> - <h4><s:text name="message.survey.merdunord.sudmerdunord" /></h4> + + <h4><s:text name="message.survey.merdunord.sudmerdunord" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.merdunord.sudmerdunord.desc" /></p> <p> <s:text name="message.survey.merdunord.sudmerdunord.plus" /> @@ -125,7 +128,9 @@ <a name="fmancheorientale"></a> <h3><s:text name="message.survey.mancheorientale" /></h3> <a name="mancheorientale"></a> - <h4><s:text name="message.survey.mancheorientale.mancheorientale" /></h4> + <h4><s:text name="message.survey.mancheorientale.mancheorientale" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.mancheorientale.mancheorientale.desc" /></p> <p> <s:text name="message.survey.mancheorientale.mancheorientale.plus" /> @@ -135,7 +140,9 @@ </p> <a name="baiedesomme"></a> - <h4><s:text name="message.survey.mancheorientale.baiedesomme" /></h4> + <h4><s:text name="message.survey.mancheorientale.baiedesomme" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.mancheorientale.baiedesomme.desc" /></p> <p> <s:text name="message.survey.mancheorientale.baiedesomme.plus" /> @@ -148,7 +155,9 @@ <a name="mancheoccidentale"></a> <h3><s:text name="message.survey.mancheoccidentale" /></h3> <a name="flamanville"></a> - <h4><s:text name="message.survey.mancheoccidentale.flamanville" /></h4> + <h4><s:text name="message.survey.mancheoccidentale.flamanville" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.mancheoccidentale.flamanville.desc" /></p> <p> <s:text name="message.survey.mancheoccidentale.flamanville.plus" /> @@ -161,7 +170,9 @@ <a name="atlantique"></a> <h3><s:text name="message.survey.atlantique" /></h3> <a name="celtique"></a> - <h4><s:text name="message.survey.atlantique.celtique" /></h4> + <h4><s:text name="message.survey.atlantique.celtique" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.atlantique.celtique.desc" /></p> <p> <s:text name="message.survey.atlantique.celtique.plus" /> @@ -171,7 +182,9 @@ </p> <a name="gascogne"></a> - <h4><s:text name="message.survey.atlantique.gascogne" /></h4> + <h4><s:text name="message.survey.atlantique.gascogne" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.atlantique.gascogne.desc" /></p> <p> <s:text name="message.survey.atlantique.gascogne.plus" /> @@ -181,7 +194,9 @@ </p> <a name="vilaine"></a> - <h4><s:text name="message.survey.atlantique.vilaine" /></h4> + <h4><s:text name="message.survey.atlantique.vilaine" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.atlantique.vilaine.desc" /></p> <p> <s:text name="message.survey.atlantique.vilaine.plus" /> @@ -194,7 +209,9 @@ <a name="mediterranee"></a> <h3><s:text name="message.survey.mediterranee" /></h3> <a name="golfelion"></a> - <h4><s:text name="message.survey.mediterranee.golfelion" /></h4> + <h4><s:text name="message.survey.mediterranee.golfelion" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.mediterranee.golfelion.desc" /></p> <p> <s:text name="message.survey.mediterranee.golfelion.plus" /> @@ -208,7 +225,9 @@ </p> <a name="estcorse"></a> - <h4><s:text name="message.survey.mediterranee.estcorse" /></h4> + <h4><s:text name="message.survey.mediterranee.estcorse" /> + <div style="float:right;text-align:right;font-weight:normal"><a href="#top"><s:text name="message.common.anchortop" /></a></div> + </h4> <p><s:text name="message.survey.mediterranee.estcorse.desc" /></p> <p> <s:text name="message.survey.mediterranee.estcorse.plus" />

1 0

r630 - trunk/coser-business/src/test/java/fr/ifremer/coser/services
by chatellier＠users.labs.libre-entreprise.org 01 Feb '11

01 Feb '11

Author: chatellier Date: 2011-02-01 09:37:07 +0000 (Tue, 01 Feb 2011) New Revision: 630 Log: Fix tests Modified: trunk/coser-business/src/test/java/fr/ifremer/coser/services/WebServiceTest.java Modified: trunk/coser-business/src/test/java/fr/ifremer/coser/services/WebServiceTest.java =================================================================== --- trunk/coser-business/src/test/java/fr/ifremer/coser/services/WebServiceTest.java 2011-02-01 09:36:52 UTC (rev 629) +++ trunk/coser-business/src/test/java/fr/ifremer/coser/services/WebServiceTest.java 2011-02-01 09:37:07 UTC (rev 630) @@ -154,8 +154,8 @@ File zip = webService.getSourceZip("testzone1", "en"); File tempDir = FileUtil.createTempDirectory("coser", ".tmp"); ZipUtil.uncompress(zip, tempDir); - Assert.assertTrue(new File(tempDir, "projet1" + File.separator + "testcatch.csv").isFile()); - Assert.assertTrue(new File(tempDir, "projet1" + File.separator + "DechargeDonnees.pdf").isFile()); + Assert.assertTrue(new File(tempDir, "RSUFI_DATA_projet1" + File.separator + "testcatch.csv").isFile()); + Assert.assertTrue(new File(tempDir, "RSUFI_DATA_projet1" + File.separator + "DechargeDonnees.pdf").isFile()); FileUtils.deleteDirectory(tempDir); }

1 0

r629 - in trunk: . coser-business/src/main/java/fr/ifremer/coser/services coser-business/src/test/resources/web
by chatellier＠users.labs.libre-entreprise.org 01 Feb '11

01 Feb '11

Author: chatellier Date: 2011-02-01 09:36:52 +0000 (Tue, 01 Feb 2011) New Revision: 629 Log: Fix pdf generation containing non escaped html entities Modified: trunk/changelog.txt trunk/coser-business/src/main/java/fr/ifremer/coser/services/WebService.java trunk/coser-business/src/test/resources/web/upload1.zip trunk/coser-business/src/test/resources/web/upload2.zip Modified: trunk/changelog.txt =================================================================== --- trunk/changelog.txt 2011-02-01 09:32:33 UTC (rev 628) +++ trunk/changelog.txt 2011-02-01 09:36:52 UTC (rev 629) @@ -4,6 +4,7 @@ 1.0.0-beta3 ----------- + * Fix pdf generation containing non escaped html entities * List projects names sorted alphabetically * Fix website texts * Close extract/upload view after extraction/upload @@ -15,7 +16,7 @@ * FileChooser opened to projectsDirectory * Update jaxx to 2.3 * Update nuiton-widgets to 1.1.1 - * Update nuiton-i18n to 2.2 + * Update nuiton-i18n to 2.3 1.0.0-beta2 ----------- Modified: trunk/coser-business/src/main/java/fr/ifremer/coser/services/WebService.java =================================================================== --- trunk/coser-business/src/main/java/fr/ifremer/coser/services/WebService.java 2011-02-01 09:32:33 UTC (rev 628) +++ trunk/coser-business/src/main/java/fr/ifremer/coser/services/WebService.java 2011-02-01 09:36:52 UTC (rev 629) @@ -54,6 +54,7 @@ import org.apache.commons.collections.map.MultiKeyMap; import org.apache.commons.io.FileUtils; import org.apache.commons.io.IOUtils; +import org.apache.commons.lang.StringEscapeUtils; import org.apache.commons.lang.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; @@ -2007,7 +2008,7 @@ Integer iMaxNumSys = Integer.valueOf(tuple[3]); specyTypes.put(specyTypeCode, new Integer[]{iMinNumSys, iMaxNumSys}); } - + // le fichier estpopind File estComIndFile = new File(resultDirectory, rsufiResult.getEstComIndName()); @@ -2035,7 +2036,7 @@ indicatorList = new TreeSet<String>(); indicatorMap.put(listNumber, indicatorList); } - + // get indicator full name String indicatorName = (String)getIndicatorsMap().get(indicatorCode, locale); // peut arriver pour les indicateurs inconnu par coser @@ -2043,7 +2044,7 @@ indicatorList.add(indicatorName); } } - + // seconde pass, remplit la map speciesMap avec les listes configurées // dans la selection for (String listNumber : indicatorMap.keySet()) { @@ -2068,6 +2069,9 @@ // get species full name String speciesName = projectService.getSpeciesDisplayName(project, speciesCode); + // fix html entities bug + speciesName = StringEscapeUtils.escapeHtml(speciesName); + // recupere le code type de l'espece, "m", "c", "p" ... Integer speciesNumSys = refTaxSpecies.get(speciesCode); for (Map.Entry<String, Integer[]> speciesTypeEntry : specyTypes.entrySet()) { Modified: trunk/coser-business/src/test/resources/web/upload1.zip =================================================================== (Binary files differ) Modified: trunk/coser-business/src/test/resources/web/upload2.zip =================================================================== (Binary files differ)

1 0

r628 - trunk
by chatellier＠users.labs.libre-entreprise.org 01 Feb '11

01 Feb '11

Author: chatellier Date: 2011-02-01 09:32:33 +0000 (Tue, 01 Feb 2011) New Revision: 628 Log: Update i18n Modified: trunk/pom.xml Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2011-01-31 14:12:53 UTC (rev 627) +++ trunk/pom.xml 2011-02-01 09:32:33 UTC (rev 628) @@ -314,7 +314,7 @@  <jaxx.version>2.3</jaxx.version> - <i18n.version>2.2</i18n.version> + <i18n.version>2.3</i18n.version> <struts.version>2.2.1.1</struts.version> </properties>

1 0