Author: fdesbois Date: 2010-12-24 12:42:45 +0000 (Fri, 24 Dec 2010) New Revision: 858 Log: - migration to Tapestry 5.2 : some actions are broken, see ticket #2607 - testify not migrated on stable version for Tapestry 5.2, tests are ignored for the moment - use proper select without zoneUpdater and subForm !!!!!! Introduced in Tapestry 5.2 !!!!!! Modified: trunk/pom.xml trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Administration.java trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Cartography.java trunk/wao-ui/src/main/webapp/Administration.tml trunk/wao-ui/src/main/webapp/Cartography.tml trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/AdministrationTest.java trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/ConnectionTest.java trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/ContactsTest.java Modified: trunk/pom.xml =================================================================== --- trunk/pom.xml 2010-12-24 11:02:31 UTC (rev 857) +++ trunk/pom.xml 2010-12-24 12:42:45 UTC (rev 858) @@ -94,12 +94,12 @@ <dependency> <groupId>com.orientimport</groupId> <artifactId>t5-easy-fckeditor</artifactId> - <version>1.0.4</version> + <version>1.0.5</version> </dependency> <dependency> <groupId>org.chenillekit</groupId> <artifactId>chenillekit-tapestry</artifactId> - <version>1.2.0</version> + <version>1.3.0</version> <exclusions> <exclusion> <groupId>jboss</groupId> @@ -333,7 +333,7 @@ <nuitonWebVersion>0.1</nuitonWebVersion> <topiaVersion>2.5-SNAPSHOT</topiaVersion> <eugeneVersion>2.3-SNAPSHOT</eugeneVersion> - <tapestryVersion>5.1.0.5</tapestryVersion> + <tapestryVersion>5.2.4</tapestryVersion> <hibernateVersion>3.3.2.GA</hibernateVersion> <jfreeVersion>1.0.12</jfreeVersion> Modified: trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Administration.java =================================================================== --- trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Administration.java 2010-12-24 11:02:31 UTC (rev 857) +++ trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Administration.java 2010-12-24 12:42:45 UTC (rev 858) @@ -32,21 +32,10 @@ import fr.ifremer.wao.bean.GlobalIndicatorValue; import fr.ifremer.wao.bean.GlobalSynthesisParameters; import fr.ifremer.wao.bean.UserRole; -import fr.ifremer.wao.entity.Company; -import fr.ifremer.wao.entity.CompanyImpl; -import fr.ifremer.wao.entity.Indicator; -import fr.ifremer.wao.entity.IndicatorLevel; -import fr.ifremer.wao.entity.UserProfile; -import fr.ifremer.wao.entity.UserProfileImpl; -import fr.ifremer.wao.entity.WaoUser; +import fr.ifremer.wao.entity.*; import fr.ifremer.wao.io.ImportResults; import fr.ifremer.wao.io.ImportResultsImpl; -import fr.ifremer.wao.service.ServiceBoat; -import fr.ifremer.wao.service.ServiceCartography; -import fr.ifremer.wao.service.ServiceReferential; -import fr.ifremer.wao.service.ServiceSampling; -import fr.ifremer.wao.service.ServiceSynthesis; -import fr.ifremer.wao.service.ServiceUser; +import fr.ifremer.wao.service.*; import fr.ifremer.wao.ui.components.Layout; import fr.ifremer.wao.ui.data.ErrorReport; import fr.ifremer.wao.ui.data.GenericSelectModel; @@ -59,14 +48,7 @@ import org.apache.tapestry5.OptionModel; import org.apache.tapestry5.RenderSupport; import org.apache.tapestry5.SelectModel; -import org.apache.tapestry5.annotations.Environmental; -import org.apache.tapestry5.annotations.IncludeJavaScriptLibrary; -import org.apache.tapestry5.annotations.IncludeStylesheet; -import org.apache.tapestry5.annotations.InjectComponent; -import org.apache.tapestry5.annotations.Log; -import org.apache.tapestry5.annotations.Persist; -import org.apache.tapestry5.annotations.Property; -import org.apache.tapestry5.annotations.SessionState; +import org.apache.tapestry5.annotations.*; import org.apache.tapestry5.beaneditor.BeanModel; import org.apache.tapestry5.beaneditor.Validate; import org.apache.tapestry5.corelib.components.BeanEditForm; @@ -470,6 +452,11 @@ log.debug("User exist in selectModel : " + userId); } userEdited = findUser(userId); + if (log.isDebugEnabled()) { + log.debug(">>User : " + userEdited); + log.debug(">>User.active : " + userEdited.isActive()); + log.debug(">>User.login : " + userEdited.getLogin()); + } } else { userEdited = serviceUser.getNewUser(getCompany()); generatePassword = true; @@ -518,6 +505,8 @@ Object onSuccessFromUserForm() throws WaoException { if (log.isDebugEnabled()) { log.debug("User : " + userEdited); + log.debug("User.active : " + userEdited.isActive()); + log.debug("User.login : " + userEdited.getLogin()); } if (!StringUtils.isEmpty(password)) { @@ -607,12 +596,13 @@ } @Log - void onChangeFromUserRole(String value) { + Zone onValueChangedFromUserRole(String value) { if (StringUtils.isEmpty(value)) { getEditedProfile().setUserRole(null); } else { getEditedProfile().setUserRole(UserRole.valueOf(value)); } + return userRoleZone; } @Log @@ -636,7 +626,7 @@ } @Log - Object onActionFromAddRole() { + Zone onActionFromAddRole() { if (getEditedProfile() != null && getEditedProfile().getUserRoleOrdinal() != null) { getEditedProfile().setCanWrite( ! readOnly); @@ -649,12 +639,12 @@ editedProfile = null; readOnly = false; // Refresh the zone - refreshUserRoleZone = true; +// refreshUserRoleZone = true; return userRoleZone; } @Log - Object onActionFromRemoveRole(int roleIndex) { + Zone onActionFromRemoveRole(int roleIndex) { if (log.isDebugEnabled()) { log.debug("edited profiles before remove : " + getEditedUserProfiles()); } @@ -666,7 +656,7 @@ } // Refresh the zone - refreshUserRoleZone = true; +// refreshUserRoleZone = true; return userRoleZone; } Modified: trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Cartography.java =================================================================== --- trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Cartography.java 2010-12-24 11:02:31 UTC (rev 857) +++ trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Cartography.java 2010-12-24 12:42:45 UTC (rev 858) @@ -20,22 +20,14 @@ */ package fr.ifremer.wao.ui.pages; -import fr.ifremer.wao.bean.ConnectedUser; -import fr.ifremer.wao.bean.ContactFilter; -import fr.ifremer.wao.bean.ContactFilterImpl; -import fr.ifremer.wao.bean.UserRole; -import fr.ifremer.wao.bean.WaoFilter; +import fr.ifremer.wao.bean.*; import fr.ifremer.wao.entity.Company; import fr.ifremer.wao.service.ServiceSampling; import fr.ifremer.wao.service.ServiceUser; import fr.ifremer.wao.ui.data.FiltersModel; import fr.ifremer.wao.ui.data.RequiresAuthentication; import org.apache.tapestry5.Link; -import org.apache.tapestry5.annotations.InjectComponent; -import org.apache.tapestry5.annotations.Log; -import org.apache.tapestry5.annotations.Persist; -import org.apache.tapestry5.annotations.Property; -import org.apache.tapestry5.annotations.SessionState; +import org.apache.tapestry5.annotations.*; import org.apache.tapestry5.corelib.components.Zone; import org.apache.tapestry5.ioc.annotations.Inject; import org.apache.tapestry5.ioc.services.PropertyAccess; @@ -129,23 +121,15 @@ @InjectComponent private Zone sampleRowFilterZone; - @Property - private boolean refreshSamplingZone; - - @Property - private boolean refreshSampleRowZone; - @Log - Zone onChangeFromCompany(String companyId) { + Zone onValueChangedFromCompany(String companyId) { getFiltersModel().updateCompany(companyId); - refreshSamplingZone = true; return samplingFilterZone; } @Log - Zone onChangeFromProgramName(String programName) { + Zone onValueChangedFromProgramName(String programName) { getFiltersModel().updateProgram(programName); - refreshSampleRowZone = true; return sampleRowFilterZone; } Modified: trunk/wao-ui/src/main/webapp/Administration.tml =================================================================== --- trunk/wao-ui/src/main/webapp/Administration.tml 2010-12-24 11:02:31 UTC (rev 857) +++ trunk/wao-ui/src/main/webapp/Administration.tml 2010-12-24 12:42:45 UTC (rev 858) @@ -115,10 +115,10 @@ </p:password> <p:userRole> <t:zone id="so-admin-userRoleZone" t:id="userRoleZone" t:update="show"> - <t:nuiton.subForm t:visible="refreshUserRoleZone"> + <!--<t:nuiton.subForm t:visible="refreshUserRoleZone">--> <label t:type="label" for="userRole" /> <select t:type="select" t:id="userRole" t:value="role" t:model="userRoleSelectModel" - t:mixins="ck/onEvent" t:event="change"/> + t:zone="so-admin-userRoleZone"/> <span>Lecture seule : </span> <input t:type="checkbox" t:id="readOnly" t:value="readOnly" @@ -135,7 +135,7 @@ </a> </li> </ul> - </t:nuiton.subForm> + <!--</t:nuiton.subForm>--> </t:zone> </p:userRole> </form> Modified: trunk/wao-ui/src/main/webapp/Cartography.tml =================================================================== --- trunk/wao-ui/src/main/webapp/Cartography.tml 2010-12-24 11:02:31 UTC (rev 857) +++ trunk/wao-ui/src/main/webapp/Cartography.tml 2010-12-24 12:42:45 UTC (rev 858) @@ -66,25 +66,25 @@ <!-- COMPANY --> <p class="mtop5"><label for="company">Société</label> : </p> <p><input t:type="select" t:id="company" t:model="filtersModel.companySelectModel" value="filtersModel.companyId" - t:mixins="nuiton/zoneUpdater" t:event="change" t:zone="cartography-filters-sampling" /></p> + t:zone="cartography-filters-sampling" /></p> </t:if> <t:zone t:id="samplingFilterZone" id="cartography-filters-sampling" t:update="show"> - <t:nuiton.subForm t:visible="refreshSamplingZone"> + <!--<t:nuiton.subForm t:visible="refreshSamplingZone">--> <!-- PROGRAM --> <p class="mtop5"><label for="programName">Programme</label> : </p> <p><select t:type="select" t:id="programName" t:model="filtersModel.programSelectModel" value="filter.programName" - t:mixins="nuiton/zoneUpdater" t:event="change" t:zone="cartography-filters-sampleRow" /></p> + t:zone="cartography-filters-sampleRow" /></p> <t:zone t:id="sampleRowFilterZone" id="cartography-filters-sampleRow" t:update="show"> - <t:nuiton.subForm t:visible="refreshSampleRowZone"> + <!--<t:nuiton.subForm t:visible="refreshSampleRowZone">--> <!-- SAMPLE_ROW --> <p class="mtop5"><label for="sampleRow">Ligne du plan</label> : </p> <p><input t:type="select" t:id="sampleRow" t:model="filtersModel.sampleRowSelectModel" value="filtersModel.sampleRowId" /></p> - </t:nuiton.subForm> + <!--</t:nuiton.subForm>--> </t:zone> - </t:nuiton.subForm> + <!--</t:nuiton.subForm>--> </t:zone> <p class="mtop5"> Modified: trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/AdministrationTest.java =================================================================== --- trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/AdministrationTest.java 2010-12-24 11:02:31 UTC (rev 857) +++ trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/AdministrationTest.java 2010-12-24 12:42:45 UTC (rev 858) @@ -24,6 +24,7 @@ import org.apache.tapestry5.dom.Document; import org.apache.tapestry5.dom.Element; import org.junit.Assert; +import org.junit.Ignore; import org.junit.Test; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -37,6 +38,8 @@ * @author fdesbois * $Id$ */ +// FIXME-fdesbois-2010-12-23 : waiting for release version of Testify for Tapestry 5.2 + at Ignore public class AdministrationTest extends AbstractApplicationTest { private static final Logger logger = Modified: trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/ConnectionTest.java =================================================================== --- trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/ConnectionTest.java 2010-12-24 11:02:31 UTC (rev 857) +++ trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/ConnectionTest.java 2010-12-24 12:42:45 UTC (rev 858) @@ -40,6 +40,7 @@ import org.apache.tapestry5.ioc.annotations.Inject; import org.apache.tapestry5.services.ApplicationStateManager; import org.junit.Assert; +import org.junit.Ignore; import org.junit.Test; import org.mockito.Mock; import org.slf4j.Logger; @@ -56,6 +57,8 @@ * * @author fdesbois <fdesbois at codelutin.com> */ +// FIXME-fdesbois-2010-12-23 : waiting for release version of Testify for Tapestry 5.2 + at Ignore public class ConnectionTest extends AbstractApplicationTest { private static final Logger logger = Modified: trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/ContactsTest.java =================================================================== --- trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/ContactsTest.java 2010-12-24 11:02:31 UTC (rev 857) +++ trunk/wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages/ContactsTest.java 2010-12-24 12:42:45 UTC (rev 858) @@ -26,6 +26,7 @@ import fr.ifremer.wao.entity.Contact; import fr.ifremer.wao.entity.WaoUser; import fr.ifremer.wao.service.ServiceContact; +import org.junit.Ignore; import org.junit.Test; import org.mockito.Mock; import org.mockito.Mockito; @@ -44,6 +45,8 @@ * @author fdesbois * $Id$ */ +// FIXME-fdesbois-2010-12-23 : waiting for release version of Testify for Tapestry 5.2 + at Ignore public class ContactsTest extends AbstractApplicationTest { private static final Logger logger =