- Wao-commits - mailman.cloud.codelutin.com

[Suiviobsmer-commits] r863 - in trunk: wao-business/src/main/java/fr/ifremer/wao/service wao-business/src/test/java/fr/ifremer/wao wao-business/src/test/java/fr/ifremer/wao/service wao-ui/src/main/java/fr/ifremer/wao/ui/pages wao-ui/src/main/webapp
by bleny＠users.labs.libre-entreprise.org Dec. 28, 2010

Dec. 28, 2010

Author: bleny Date: 2010-12-28 09:56:05 +0000 (Tue, 28 Dec 2010) New Revision: 863 Log: fix multiple observers form Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceContactImpl.java trunk/wao-business/src/test/java/fr/ifremer/wao/AbstractServiceTest.java trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceContactImplTest.java trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/ContactForm.java trunk/wao-ui/src/main/webapp/ContactForm.tml Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceContactImpl.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceContactImpl.java 2010-12-27 14:29:24 UTC (rev 862) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceContactImpl.java 2010-12-28 09:56:05 UTC (rev 863) @@ -333,6 +333,9 @@ Contact.PROPERTY_SAMPLE_ROW ); + destination.clearSecondaryObservers(); + destination.addAllSecondaryObservers(source.getSecondaryObservers()); + // Update the change dao.update(destination); } Modified: trunk/wao-business/src/test/java/fr/ifremer/wao/AbstractServiceTest.java =================================================================== --- trunk/wao-business/src/test/java/fr/ifremer/wao/AbstractServiceTest.java 2010-12-27 14:29:24 UTC (rev 862) +++ trunk/wao-business/src/test/java/fr/ifremer/wao/AbstractServiceTest.java 2010-12-28 09:56:05 UTC (rev 863) @@ -23,8 +23,24 @@ import fr.ifremer.wao.bean.ContactState; import fr.ifremer.wao.bean.UserRole; -import fr.ifremer.wao.entity.*; -import fr.ifremer.wao.service.*; +import fr.ifremer.wao.entity.Boat; +import fr.ifremer.wao.entity.BoatDAO; +import fr.ifremer.wao.entity.Company; +import fr.ifremer.wao.entity.CompanyDAO; +import fr.ifremer.wao.entity.Contact; +import fr.ifremer.wao.entity.ContactDAO; +import fr.ifremer.wao.entity.ContactImpl; +import fr.ifremer.wao.entity.ElligibleBoat; +import fr.ifremer.wao.entity.ElligibleBoatDAO; +import fr.ifremer.wao.entity.SampleRow; +import fr.ifremer.wao.entity.SampleRowDAO; +import fr.ifremer.wao.entity.WaoUser; +import fr.ifremer.wao.entity.WaoUserDAO; +import fr.ifremer.wao.service.ServiceBoat; +import fr.ifremer.wao.service.ServiceContact; +import fr.ifremer.wao.service.ServiceReferential; +import fr.ifremer.wao.service.ServiceSampling; +import fr.ifremer.wao.service.ServiceSamplingImpl; import org.junit.After; import org.junit.Before; import org.junit.BeforeClass; Modified: trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceContactImplTest.java =================================================================== --- trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceContactImplTest.java 2010-12-27 14:29:24 UTC (rev 862) +++ trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceContactImplTest.java 2010-12-28 09:56:05 UTC (rev 863) @@ -8,12 +8,12 @@ * it under the terms of the GNU Affero General Public License as published by * the Free Software Foundation, either version 3 of the License, or * (at your option) any later version. - * + * * This program is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. - * + * * You should have received a copy of the GNU Affero General Public License * along with this program. If not, see <http://www.gnu.org/licenses/>. * #L% @@ -26,6 +26,7 @@ import fr.ifremer.wao.bean.ContactFilter; import fr.ifremer.wao.bean.ContactFilterImpl; import fr.ifremer.wao.bean.UserRole; +import fr.ifremer.wao.entity.Company; import fr.ifremer.wao.entity.Contact; import fr.ifremer.wao.entity.ContactImpl; import fr.ifremer.wao.entity.WaoUser; @@ -80,26 +81,46 @@ log.info("test 2 : Bad filter, no contact is returned"); WaoUser observer2 = createUser("marge", UserRole.OBSERVER, findDefaultCompany()); - + filter.setObserver(observer2); results = service.getContacts(filter); Assert.assertEquals(0, results.size()); } - @Test - public void testSaveContact() throws TopiaException { + @Test(expected = NullPointerException.class) + public void failOnSavingContactWithNullSampleRow() throws Throwable { Contact contact = new ContactImpl(); WaoUser observer = createDefaultObserver(); contact.setMainObserver(observer); - try { service.saveContact(contact, false); Assert.fail(); } catch (WaoException eee) { - + throw eee.getCause(); } } + + @Test + public void testSaveContactWithMultipleObservers() throws TopiaException { + Contact contact = createDefaultContact(); + + Company company = createCompany("Boite"); + contact.addSecondaryObservers(createUser("toto", UserRole.OBSERVER, company)); + contact.addSecondaryObservers(createUser("titi", UserRole.OBSERVER, company)); + + service.saveContact(contact, false); + + Contact savedContact = service.getContact(contact.getTopiaId()); + Assert.assertEquals(2, savedContact.getSecondaryObservers().size()); + + contact.clearSecondaryObservers(); + service.saveContact(contact, false); + savedContact = service.getContact(contact.getTopiaId()); + Assert.assertEquals(0, savedContact.getSecondaryObservers().size()); + } + + } Modified: trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/ContactForm.java =================================================================== --- trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/ContactForm.java 2010-12-27 14:29:24 UTC (rev 862) +++ trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/ContactForm.java 2010-12-28 09:56:05 UTC (rev 863) @@ -479,7 +479,8 @@ if (logger.isDebugEnabled()) { logger.debug("Contact save : " + contact); - // logger.debug("Contact sendEmail : " + sendEmail); + logger.debug("secondary observers : " + contact.getSecondaryObservers()); + logger.debug("Contact sendEmail : " + sendEmail); } serviceContact.saveContact(contact, false); Modified: trunk/wao-ui/src/main/webapp/ContactForm.tml =================================================================== --- trunk/wao-ui/src/main/webapp/ContactForm.tml 2010-12-27 14:29:24 UTC (rev 862) +++ trunk/wao-ui/src/main/webapp/ContactForm.tml 2010-12-28 09:56:05 UTC (rev 863) @@ -77,25 +77,23 @@ <input t:type="select" t:id="mainObserver" t:model="observerSelectModel" t:validate="required" t:value="mainObserverId" /> </div> <t:zone t:id="observersFormZone" t:visible="true" t:update="show"> - <t:nuiton.subForm t:visible="refreshObserversForm"> - <div class="t-beaneditor-row"> - <t:label t:for="secondaryObserver" /> - <input t:type="select" t:id="secondaryObserver" t:model="observerSelectModel" t:value="selectedSecondaryObserverId" t:mixins="ck/onEvent" t:event="change" /> - <a t:type="actionlink" t:id="addSecondaryObserver" title="Ajouter un observateur" t:zone="observersFormZone"> - <img src="${asset:context:img/add-16px.png}" /> - </a>  - <div class="t-beaneditor-row" style="position: relative; left: 250px;"> - <ul> - <li t:type="loop" t:source="secondaryObservers" t:value="observer" t:index="secondaryObserverIndex" t:volatile="true"> - ${observer.fullName} - <a t:type="actionlink" t:id="removeSecondaryObserver" t:context="secondaryObserverIndex" title="Supprimer cet observateur" t:zone="observersFormZone"> - <img src="${asset:context:img/remove-22px.png}" /> - </a> - </li> - </ul> - </div> + <div class="t-beaneditor-row"> + <t:label t:for="secondaryObserver" /> + <input t:type="select" t:id="secondaryObserver" t:model="observerSelectModel" t:value="selectedSecondaryObserverId" t:mixins="ck/onEvent" t:event="change" /> + <a t:type="actionlink" t:id="addSecondaryObserver" title="Ajouter un observateur" t:zone="observersFormZone"> + <img src="${asset:context:img/add-16px.png}" /> + </a>  + <div class="t-beaneditor-row" style="position: relative; left: 250px;"> + <ul> + <li t:type="loop" t:source="secondaryObservers" t:value="observer" t:index="secondaryObserverIndex" t:volatile="true"> + ${observer.fullName} + <a t:type="actionlink" t:id="removeSecondaryObserver" t:context="secondaryObserverIndex" title="Supprimer cet observateur" t:zone="observersFormZone"> + <img src="${asset:context:img/remove-22px.png}" /> + </a> + </li> + </ul> </div> - </t:nuiton.subForm> + </div> </t:zone> </t:block> @@ -168,7 +166,7 @@ </div> <div class="t-beaneditor-row"> <t:label t:for="nbObservants" /> - <input t:type="textfield" t:id="nbObservants" t:value="contact.nbObservants" t:validate="required"/> + <input t:type="textfield" t:id="nbObservants" t:value="contact.nbObservants" t:validate="required" size="2" /> </div> <div class="t-beaneditor-row"> <t:label t:for="beginDate" />

1 0

[Suiviobsmer-commits] r862 - in trunk/wao-ui/src/main: java/fr/ifremer/wao/ui/components resources/fr/ifremer/wao/ui/components webapp/css
by bleny＠users.labs.libre-entreprise.org Dec. 27, 2010

Dec. 27, 2010

Author: bleny Date: 2010-12-27 14:29:24 +0000 (Mon, 27 Dec 2010) New Revision: 862 Log: refactor menu generation depending on profile Modified: trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/components/Layout.java trunk/wao-ui/src/main/resources/fr/ifremer/wao/ui/components/Layout.tml trunk/wao-ui/src/main/webapp/css/main.css Modified: trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/components/Layout.java =================================================================== --- trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/components/Layout.java 2010-12-24 16:32:31 UTC (rev 861) +++ trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/components/Layout.java 2010-12-27 14:29:24 UTC (rev 862) @@ -25,7 +25,12 @@ package fr.ifremer.wao.ui.components; import fr.ifremer.wao.bean.ConnectedUser; +import fr.ifremer.wao.ui.pages.Administration; +import fr.ifremer.wao.ui.pages.Boats; +import fr.ifremer.wao.ui.pages.Contacts; import fr.ifremer.wao.ui.pages.Index; +import fr.ifremer.wao.ui.pages.SamplingPlan; +import fr.ifremer.wao.ui.pages.Synthesis; import fr.ifremer.wao.ui.services.ServiceAuthentication; import fr.ifremer.wao.ui.services.WaoManager; import org.apache.tapestry5.BindingConstants; @@ -38,6 +43,9 @@ import org.apache.tapestry5.services.Request; import org.slf4j.Logger; +import java.util.LinkedHashMap; +import java.util.Map; + /** * Layout * @@ -122,6 +130,29 @@ return this.contentFeedback; } + /*************************** MAIN MENU ************************************/ + + /** Page links and link title. */ + private Map<String, String> menuItems; + + @Property + private Map.Entry<String, String> menuItem; + + public Map getMenuItems() { + if (menuItems == null) { + Class[] pagesInMenu = new Class[] { SamplingPlan.class, Boats.class, + Contacts.class, Synthesis.class, Administration.class}; + menuItems = new LinkedHashMap<String, String>(); + for (Class page : pagesInMenu) { + if (serviceAuthentication.isAllowed(page)) { + // menuItems.put(page.getSimpleName(), _(page.getName())); + menuItems.put(page.getSimpleName(), page.getSimpleName()); + } + } + } + return menuItems; + } + public String getAdminClass() { return !isAdministrationHidden() ? " admin" : ""; } Modified: trunk/wao-ui/src/main/resources/fr/ifremer/wao/ui/components/Layout.tml =================================================================== --- trunk/wao-ui/src/main/resources/fr/ifremer/wao/ui/components/Layout.tml 2010-12-24 16:32:31 UTC (rev 861) +++ trunk/wao-ui/src/main/resources/fr/ifremer/wao/ui/components/Layout.tml 2010-12-27 14:29:24 UTC (rev 862) @@ -83,7 +83,14 @@ </a> </div> </div> + <ul class="clearfix" id="so-menu"> + <li t:type="loop" source="menuItems.entrySet()" value="menuItem"> + <a href="${contextPath}/${menuItem.key}">${menuItem.value}</a> + </li> + </ul> + +  + <!- Menu disabled for Guest user -> <li class="disabled" title="Liste des navires inaccessible"> Navires </li> @@ -129,7 +136,7 @@ </p:else> </t:if> </t:unless> - </ul> + </ul--> <div id="${contentId}"> <t:feedback t:id="contentFeedback" t:autoClear="false"/> Modified: trunk/wao-ui/src/main/webapp/css/main.css =================================================================== --- trunk/wao-ui/src/main/webapp/css/main.css 2010-12-24 16:32:31 UTC (rev 861) +++ trunk/wao-ui/src/main/webapp/css/main.css 2010-12-27 14:29:24 UTC (rev 862) @@ -137,65 +137,39 @@ padding-right: 40px; } -/* MENU */ -ul#so-menu { - width: 80%; - height: 40px; +#so-menu { + width: 800px; + margin-top: 7px; margin-left: auto; margin-right: auto; - margin-top: 7px; + display: table; } -ul#so-menu li { - list-style-type: none; +#so-menu li { + display: table-cell; text-align: center; - float:left; + padding: 0px; } -ul#so-menu li.link { - background-color: #DEE7EC; - padding: 5px; - width: 23%; +#so-menu li:first-child { + margin-left: 0px; } -ul#so-menu li.disabled { - background-color: #e5e5e5; - color: white; - padding: 5px; - width: 23%; +#so-menu a { + color: black; + display: block; + width: inherit; + background-color: #DEE7EC; + padding: 7px; + text-decoration: none; + margin-left: 15px; } -ul#so-menu li.admin { - width: 18% !important; -} - -/* Separation */ -ul#so-menu li.sep { - height: 12px; - width: 0.8%; - background-color: #007CC2; -} -/* Home */ -ul#so-menu li.home { - padding: 4px 0 0 2px; - width: 25px; -} - -ul#so-menu li.selected { +#so-menu a:hover, #so-menu a:active, #so-menu a:focus { background-color: #133852; - border-top: 3px solid #007CC2; - border-bottom: 3px solid #007CC2; color: white; } -ul#so-menu li.link:hover { - background-color: #133852; - border-top: 3px solid #007CC2; - border-bottom: 3px solid #007CC2; - color: white; - cursor: pointer; -} - div#so-footer { margin-top: 20px; margin-bottom: 20px; @@ -216,3 +190,4 @@ div#so-footer p.cnil { font-size: 0.7em; } +

1 0

[Suiviobsmer-commits] r861 - trunk/wao-business/src/main/java/fr/ifremer/wao/bean
by bleny＠users.labs.libre-entreprise.org Dec. 24, 2010

Dec. 24, 2010

Author: bleny Date: 2010-12-24 16:32:31 +0000 (Fri, 24 Dec 2010) New Revision: 861 Log: make indicators viewable only for ObsMer (admin) Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/bean/ConnectedUserImpl.java Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/bean/ConnectedUserImpl.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/bean/ConnectedUserImpl.java 2010-12-24 15:55:29 UTC (rev 860) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/bean/ConnectedUserImpl.java 2010-12-24 16:32:31 UTC (rev 861) @@ -88,7 +88,9 @@ Boolean indicatorsAreActivated = Boolean.parseBoolean(WaoProperty.SHOW_INDICATORS.getValue()); if (BooleanUtils.isTrue(indicatorsAreActivated)) { - test = isAdmin() || isCoordinator(); + test = ObsProgram.OBSMER.equals(getProfile().getObsProgram()) + && isAdmin() || isCoordinator(); + } return test;

1 0

[Suiviobsmer-commits] r860 - trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages
by bleny＠users.labs.libre-entreprise.org Dec. 24, 2010

Dec. 24, 2010

Author: bleny Date: 2010-12-24 15:55:29 +0000 (Fri, 24 Dec 2010) New Revision: 860 Log: make profiles edition program-independant Modified: trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Administration.java 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 15:10:34 UTC (rev 859) +++ trunk/wao-ui/src/main/java/fr/ifremer/wao/ui/pages/Administration.java 2010-12-24 15:55:29 UTC (rev 860) @@ -32,10 +32,21 @@ import fr.ifremer.wao.bean.GlobalIndicatorValue; import fr.ifremer.wao.bean.GlobalSynthesisParameters; import fr.ifremer.wao.bean.UserRole; -import fr.ifremer.wao.entity.*; +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.io.ImportResults; import fr.ifremer.wao.io.ImportResultsImpl; -import fr.ifremer.wao.service.*; +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.ui.components.Layout; import fr.ifremer.wao.ui.data.ErrorReport; import fr.ifremer.wao.ui.data.GenericSelectModel; @@ -48,7 +59,14 @@ import org.apache.tapestry5.OptionModel; import org.apache.tapestry5.RenderSupport; import org.apache.tapestry5.SelectModel; -import org.apache.tapestry5.annotations.*; +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.beaneditor.BeanModel; import org.apache.tapestry5.beaneditor.Validate; import org.apache.tapestry5.corelib.components.BeanEditForm; @@ -496,7 +514,7 @@ } else { // set may be buggy // getUserEdited().setUserProfile(getEditedUserProfiles()); - getUserEdited().clearUserProfile(); + getUserEdited().clearUserProfile(currentUser.getProfile().getObsProgram()); getUserEdited().addAllUserProfile(getEditedUserProfiles()); } } @@ -616,10 +634,8 @@ @Log public List<UserProfile> getEditedUserProfiles() { if (editedUserProfiles == null) { - if (getUserEdited() != null && getUserEdited().getUserProfile() != null) { - editedUserProfiles = new ArrayList<UserProfile>(getUserEdited().getUserProfile()); - } else { - editedUserProfiles = new ArrayList<UserProfile>(); + if (getUserEdited() != null) { + editedUserProfiles = getUserEdited().getUserProfile(currentUser.getProfile().getObsProgram()); } } return editedUserProfiles;

1 0

[Suiviobsmer-commits] r859 - in trunk/wao-business/src/main: java/fr/ifremer/wao/entity java/fr/ifremer/wao/service xmi
by bleny＠users.labs.libre-entreprise.org Dec. 24, 2010

Dec. 24, 2010

Author: bleny Date: 2010-12-24 15:10:34 +0000 (Fri, 24 Dec 2010) New Revision: 859 Log: fix user update Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/entity/WaoUserImpl.java trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java trunk/wao-business/src/main/xmi/wao.zargo Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/entity/WaoUserImpl.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/entity/WaoUserImpl.java 2010-12-24 12:42:45 UTC (rev 858) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/entity/WaoUserImpl.java 2010-12-24 15:10:34 UTC (rev 859) @@ -25,13 +25,14 @@ package fr.ifremer.wao.entity; import fr.ifremer.wao.WaoUtils; +import fr.ifremer.wao.bean.ObsProgram; import fr.ifremer.wao.bean.UserRole; import org.slf4j.Logger; import org.slf4j.LoggerFactory; import java.io.Serializable; import java.util.ArrayList; -import java.util.Collection; +import java.util.List; /** * UserImpl @@ -89,4 +90,25 @@ } return false; } + + @Override + public List<UserProfile> getUserProfile(ObsProgram obsProgram) { + List<UserProfile> userProfiles = new ArrayList<UserProfile>(); + if (getUserProfile() != null) { + for (UserProfile userProfile : getUserProfile()) { + if (userProfile.getObsProgram().equals(obsProgram)) { + userProfiles.add(userProfile); + } + } + } + return userProfiles; + } + + @Override + public void clearUserProfile(ObsProgram obsProgram) { + List<UserProfile> userProfiles = getUserProfile(obsProgram); + for (UserProfile userProfile : userProfiles) { + removeUserProfile(userProfile); + } + } } Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java 2010-12-24 12:42:45 UTC (rev 858) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java 2010-12-24 15:10:34 UTC (rev 859) @@ -193,148 +193,53 @@ } @Override - public void executeCreateUpdateUser(TopiaContext transaction, - WaoUser user, boolean generatePassword) - throws TopiaException, IllegalArgumentException, - WaoBusinessException { - + protected void executeCreateUpdateUser(TopiaContext transaction, WaoUser user, boolean generatePassword) throws Exception { + if (user == null) { throw new IllegalArgumentException("user parameter can't be null"); } WaoUserDAO dao = WaoDAOHelper.getWaoUserDAO(transaction); - UserProfileDAO userProfileDAO = WaoDAOHelper.getUserProfileDAO(transaction); - WaoUser userToUpdate; + boolean isNewUser = user.getTopiaId() == null; - boolean newUser = user.getTopiaId() == null; + if (isNewUser) { - if (newUser) { - userToUpdate = user; - - - context.prepareTopiaId(WaoUser.class, userToUpdate); - WaoUser existUser = dao.findByLogin(user.getLogin()); + WaoUser existingUser = dao.findByLogin(user.getLogin()); // Check for a new user if login already exists - if (existUser != null) { + if (existingUser != null) { throw new WaoBusinessException(Type.ALREADY_EXISTS, this.getClass(), "Un utilisateur existe déjà avec ce login " + user.getLogin()); } - if (user.getUserProfile() != null) { - - Collection<UserProfile> newProfiles = new ArrayList<UserProfile>(user.getUserProfile()); - userToUpdate.clearUserProfile(); - for (UserProfile profile : newProfiles) { - log.debug("saving profile " + profile.getDescription()); - UserProfile profileToAdd = userProfileDAO.create( - UserProfile.PROPERTY_OBS_PROGRAM_ORDINAL, - profile.getObsProgramOrdinal(), - UserProfile.PROPERTY_USER_ROLE_ORDINAL, - profile.getUserRoleOrdinal(), - UserProfile.PROPERTY_CAN_WRITE, - profile.getCanWrite() - ); - userToUpdate.addUserProfile(profileToAdd); - } - } - } else { - userToUpdate = dao.findByTopiaId(user.getTopiaId()); - /* - Collection<UserProfile> profilesToRemove; - if (userToUpdate.getUserProfile() == null) { - profilesToRemove = Collections.EMPTY_LIST; - } else { - profilesToRemove = new ArrayList<UserProfile>(userToUpdate.getUserProfile()); - } + context.prepareTopiaId(WaoUser.class, user); + } - if (log.isDebugEnabled()) { - log.debug("old profiles : " + profilesToRemove); - } -*/ - // set may be buggy -// userToUpdate.setUserProfile(new ArrayList<UserProfile>(user.getUserProfile())); -// userToUpdate.clearUserProfile(); -// userToUpdate.addAllUserProfile(user.getUserProfile()); - - List<UserProfile> newProfiles = new ArrayList<UserProfile>(); - - for (UserProfile profile : user.getUserProfile()) { - UserProfile profileToAdd; - if (profile.getTopiaId() == null) { - - // WaoBinderHelper.getSimpleTopiaBinder(UserProfile.class).copy(profile, profileToAdd); - - // userProfileDAO.create(profileToAdd); - profileToAdd = userProfileDAO.create( - UserProfile.PROPERTY_OBS_PROGRAM_ORDINAL, - profile.getObsProgramOrdinal(), - UserProfile.PROPERTY_USER_ROLE_ORDINAL, - profile.getUserRoleOrdinal(), - UserProfile.PROPERTY_CAN_WRITE, - profile.getCanWrite() - ); - } else { - profileToAdd = userProfileDAO.findByTopiaId(profile.getTopiaId()); - } - newProfiles.add(profileToAdd); - } - - - userToUpdate.clearUserProfile(); - userToUpdate.addAllUserProfile(newProfiles); - -/* - if (log.isDebugEnabled()) { - log.debug("new profiles : " + userToUpdate.getUserProfile()); - } - + if (user.getUserProfile() != null) { for (UserProfile userProfile : user.getUserProfile()) { - if (log.isDebugEnabled()) { - log.debug("is " + userProfile + " in " + profilesToRemove + " ? " - + profilesToRemove.contains(userProfile)); + if (userProfile.getTopiaId() == null) { + context.prepareTopiaId(UserProfile.class, userProfile); } - if (profilesToRemove.contains(userProfile)) { - // this profile was already attributed to this user, don't - // remove it : just keep it from being removed - if (log.isDebugEnabled()) { - log.debug("profile " + userProfile + " was already given to user"); - } - profilesToRemove.remove(userProfile); - } else { - // this profile was added to the user - if (log.isDebugEnabled()) { - log.debug("profile " + userProfile + " is new for user"); - } - userProfileDAO.create(userProfile); - } } - // now, oldProfiles contains user has no longer, we must remove them - if (log.isDebugEnabled()) { - log.debug("will remove deleted profiles : " + profilesToRemove); - } - for (UserProfile userProfile : profilesToRemove) { - userProfileDAO.delete(userProfile); - }*/ } - String password = userToUpdate.getPassword(); + String password = user.getPassword(); if (generatePassword) { password = context.createRandomString(8); - userToUpdate.setPasswordChanged(true); + user.setPasswordChanged(true); } // For a password set manually by user or generated - if (userToUpdate.isPasswordChanged()) { + if (user.isPasswordChanged()) { String passwordEncoded = context.encodeString(password); - userToUpdate.setPassword(passwordEncoded); + user.setPassword(passwordEncoded); } - log.debug("will store " + userToUpdate.getTopiaId()); - dao.update(userToUpdate); - log.debug("had stored " + userToUpdate.getTopiaId()); + log.debug("will store " + user.getTopiaId()); + dao.update(user); + log.debug("had stored " + user.getTopiaId()); transaction.commitTransaction(); @@ -343,7 +248,7 @@ if (user.getLogin().contains("@") && user.isPasswordChanged()) { String subject = "[WAO] "; String msg = "Bonjour,\n\n"; - if (newUser) { + if (isNewUser) { subject += "Création de votre compte"; msg += "\tVous avez été inscris sur le site WAO :" + " Web Applicatif Obsmer. Vos identifiants de " + Modified: trunk/wao-business/src/main/xmi/wao.zargo =================================================================== (Binary files differ)

1 0

[Suiviobsmer-commits] r858 - in trunk: . wao-ui/src/main/java/fr/ifremer/wao/ui/pages wao-ui/src/main/webapp wao-ui/src/test/java/test/fr/ifremer/wao/ui/pages
by fdesbois＠users.labs.libre-entreprise.org Dec. 24, 2010

Dec. 24, 2010

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"> +  <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: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 @@  <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"> +   <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"> +   <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:zone> - </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 =

1 0

[Suiviobsmer-commits] r857 - in trunk/wao-business/src: main/java/fr/ifremer/wao/entity main/java/fr/ifremer/wao/service main/xmi test/java/fr/ifremer/wao/service
by bleny＠users.labs.libre-entreprise.org Dec. 24, 2010

Dec. 24, 2010

Author: bleny Date: 2010-12-24 11:02:31 +0000 (Fri, 24 Dec 2010) New Revision: 857 Log: some refactoring ; code-style Added: trunk/wao-business/src/main/java/fr/ifremer/wao/entity/WaoUserDAOImpl.java Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java trunk/wao-business/src/main/xmi/wao.zargo trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceUserImplTest.java Added: trunk/wao-business/src/main/java/fr/ifremer/wao/entity/WaoUserDAOImpl.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/entity/WaoUserDAOImpl.java (rev 0) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/entity/WaoUserDAOImpl.java 2010-12-24 11:02:31 UTC (rev 857) @@ -0,0 +1,18 @@ +package fr.ifremer.wao.entity; + +import org.nuiton.topia.TopiaException; +import org.nuiton.topia.framework.TopiaQuery; + +import java.util.List; + +public class WaoUserDAOImpl<E extends WaoUser> extends WaoUserDAOAbstract<E> { + + @Override + public List<WaoUser> findUsersByCompany(Company company) throws TopiaException { + TopiaQuery query = createQuery(). + addEquals(WaoUser.PROPERTY_COMPANY, company). + addOrder(WaoUser.PROPERTY_FIRST_NAME, WaoUser.PROPERTY_LAST_NAME); + + return (List<WaoUser>) findAllByQuery(query); + } +} Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java 2010-12-23 19:09:52 UTC (rev 856) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java 2010-12-24 11:02:31 UTC (rev 857) @@ -24,10 +24,36 @@ package fr.ifremer.wao.service; -import fr.ifremer.wao.*; +import fr.ifremer.wao.WaoBusinessException; import fr.ifremer.wao.WaoBusinessException.Type; -import fr.ifremer.wao.bean.*; -import fr.ifremer.wao.entity.*; +import fr.ifremer.wao.WaoContext; +import fr.ifremer.wao.WaoDAOHelper; +import fr.ifremer.wao.WaoException; +import fr.ifremer.wao.WaoProperty; +import fr.ifremer.wao.WaoQueryHelper; +import fr.ifremer.wao.bean.BoatFilter; +import fr.ifremer.wao.bean.BoatFilterImpl; +import fr.ifremer.wao.bean.ConnectedUser; +import fr.ifremer.wao.bean.ConnectedUserImpl; +import fr.ifremer.wao.bean.ObsProgram; +import fr.ifremer.wao.bean.SamplingFilter; +import fr.ifremer.wao.bean.SamplingFilterImpl; +import fr.ifremer.wao.bean.UserRole; +import fr.ifremer.wao.entity.Boat; +import fr.ifremer.wao.entity.Company; +import fr.ifremer.wao.entity.CompanyDAO; +import fr.ifremer.wao.entity.Contact; +import fr.ifremer.wao.entity.ContactDAO; +import fr.ifremer.wao.entity.News; +import fr.ifremer.wao.entity.SampleRow; +import fr.ifremer.wao.entity.SampleRowLog; +import fr.ifremer.wao.entity.SampleRowLogDAO; +import fr.ifremer.wao.entity.UserProfile; +import fr.ifremer.wao.entity.UserProfileDAO; +import fr.ifremer.wao.entity.UserProfileImpl; +import fr.ifremer.wao.entity.WaoUser; +import fr.ifremer.wao.entity.WaoUserDAO; +import fr.ifremer.wao.entity.WaoUserImpl; import org.apache.commons.mail.EmailException; import org.nuiton.topia.TopiaContext; import org.nuiton.topia.TopiaException; @@ -379,7 +405,7 @@ List<SampleRowLog> results = logDAO.findAllByAuthor(user); if (!results.isEmpty()) { - String msg = ""; + String msg; if (isCoordinator) { msg = " car il a ajouté des commentaires sur le plan " + "d'échantillonnage."; @@ -451,23 +477,11 @@ errorArgs.add(company.getName()); - List<WaoUser> users = getUsersByCompany(transaction, company); - - return users; - } - - /** - *@deprecated FIXME-fdesbois-2010-12-23 : move this method into WaoUserDAO - */ - @Deprecated - protected List<WaoUser> getUsersByCompany(TopiaContext transaction, Company company) throws TopiaException { WaoUserDAO dao = WaoDAOHelper.getWaoUserDAO(transaction); - TopiaQuery query = dao.createQuery(). - addEquals(WaoUser.PROPERTY_COMPANY, company). - addOrder(WaoUser.PROPERTY_FIRST_NAME, WaoUser.PROPERTY_LAST_NAME); + List<WaoUser> users = dao.findUsersByCompany(company); - return dao.findAllByQuery(query); + return users; } @Override @@ -575,7 +589,8 @@ protected void executeDeleteCompany(TopiaContext transaction, Company company) throws Exception { // we can delete a company only if no users are attached - List<WaoUser> users = getUsersByCompany(transaction, company); + WaoUserDAO waoUserDAO = WaoDAOHelper.getWaoUserDAO(transaction); + List<WaoUser> users = waoUserDAO.findUsersByCompany(company); if ( ! users.isEmpty()) { String message = "Impossible de supprimer la société " + company.getName() + " : des utilisateurs sont encore associés à cette société"; Modified: trunk/wao-business/src/main/xmi/wao.zargo =================================================================== (Binary files differ) Modified: trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceUserImplTest.java =================================================================== --- trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceUserImplTest.java 2010-12-23 19:09:52 UTC (rev 856) +++ trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceUserImplTest.java 2010-12-24 11:02:31 UTC (rev 857) @@ -25,12 +25,17 @@ package fr.ifremer.wao.service; import fr.ifremer.wao.AbstractServiceTest; -import fr.ifremer.wao.TestManager; import fr.ifremer.wao.WaoBusinessException; import fr.ifremer.wao.WaoDAOHelper; import fr.ifremer.wao.bean.ObsProgram; import fr.ifremer.wao.bean.UserRole; -import fr.ifremer.wao.entity.*; +import fr.ifremer.wao.entity.Company; +import fr.ifremer.wao.entity.CompanyImpl; +import fr.ifremer.wao.entity.UserProfile; +import fr.ifremer.wao.entity.UserProfileDAO; +import fr.ifremer.wao.entity.UserProfileImpl; +import fr.ifremer.wao.entity.WaoUser; +import fr.ifremer.wao.entity.WaoUserImpl; import org.junit.Assert; import org.junit.Before; import org.junit.Test; @@ -40,7 +45,6 @@ import org.slf4j.LoggerFactory; import java.util.ArrayList; -import java.util.LinkedList; import java.util.List; /** @@ -70,7 +74,7 @@ * @throws Exception all exceptions */ @Test - public void testCreateUpdateUser() throws Exception { + public void testCreateUser() throws Exception { log.info("############### START ############### testCreateUpdateUser"); // create a user with no role @@ -80,8 +84,6 @@ // now, update user with or without adding, removing, a profile to // see if an error is raised - List<UserProfile> profiles = new LinkedList<UserProfile>(); - user.setUserProfile(profiles); UserProfile adminProfile = new UserProfileImpl( ObsProgram.OBSMER, @@ -94,23 +96,21 @@ true ); - profiles.add(adminProfile); + user.addUserProfile(adminProfile); log.debug("trying to store " + user.getTopiaId()); service.createUpdateUser(user, false); - manager = new TestManager(); - setUp(); + user.addUserProfile(coordinatorProfile); + service.createUpdateUser(user, false); - profiles.add(coordinatorProfile); + user.removeUserProfile(adminProfile); service.createUpdateUser(user, false); - manager = new TestManager(); - setUp(); + WaoUser userFound = findUserByLogin("login"); + Assert.assertNotNull(userFound); + Assert.assertEquals(1, userFound.getUserProfile().size()); - profiles.remove(adminProfile); - service.createUpdateUser(user, false); - // check that creating two user with the same login is not allowed try { user = new WaoUserImpl(); @@ -159,6 +159,7 @@ WaoUser userFound = findUserByLogin(login); log.debug("user found = " + userFound); + Assert.assertNotNull(userFound); Assert.assertFalse(userFound.isActive()); }

1 0

[Suiviobsmer-commits] r856 - in trunk/wao-business/src: main/java/fr/ifremer/wao/service test/java/fr/ifremer/wao test/java/fr/ifremer/wao/service
by fdesbois＠users.labs.libre-entreprise.org Dec. 23, 2010

Dec. 23, 2010

Author: fdesbois Date: 2010-12-23 19:09:52 +0000 (Thu, 23 Dec 2010) New Revision: 856 Log: Add broken test for updateUser Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java trunk/wao-business/src/test/java/fr/ifremer/wao/AbstractServiceTest.java trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceUserImplTest.java Modified: trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java =================================================================== --- trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java 2010-12-23 17:15:46 UTC (rev 855) +++ trunk/wao-business/src/main/java/fr/ifremer/wao/service/ServiceUserImpl.java 2010-12-23 19:09:52 UTC (rev 856) @@ -24,36 +24,10 @@ package fr.ifremer.wao.service; -import fr.ifremer.wao.WaoBusinessException; +import fr.ifremer.wao.*; import fr.ifremer.wao.WaoBusinessException.Type; -import fr.ifremer.wao.WaoContext; -import fr.ifremer.wao.WaoDAOHelper; -import fr.ifremer.wao.WaoException; -import fr.ifremer.wao.WaoProperty; -import fr.ifremer.wao.WaoQueryHelper; -import fr.ifremer.wao.bean.BoatFilter; -import fr.ifremer.wao.bean.BoatFilterImpl; -import fr.ifremer.wao.bean.ConnectedUser; -import fr.ifremer.wao.bean.ConnectedUserImpl; -import fr.ifremer.wao.bean.ObsProgram; -import fr.ifremer.wao.bean.SamplingFilter; -import fr.ifremer.wao.bean.SamplingFilterImpl; -import fr.ifremer.wao.bean.UserRole; -import fr.ifremer.wao.entity.Boat; -import fr.ifremer.wao.entity.Company; -import fr.ifremer.wao.entity.CompanyDAO; -import fr.ifremer.wao.entity.Contact; -import fr.ifremer.wao.entity.ContactDAO; -import fr.ifremer.wao.entity.News; -import fr.ifremer.wao.entity.SampleRow; -import fr.ifremer.wao.entity.SampleRowLog; -import fr.ifremer.wao.entity.SampleRowLogDAO; -import fr.ifremer.wao.entity.UserProfile; -import fr.ifremer.wao.entity.UserProfileDAO; -import fr.ifremer.wao.entity.UserProfileImpl; -import fr.ifremer.wao.entity.WaoUser; -import fr.ifremer.wao.entity.WaoUserDAO; -import fr.ifremer.wao.entity.WaoUserImpl; +import fr.ifremer.wao.bean.*; +import fr.ifremer.wao.entity.*; import org.apache.commons.mail.EmailException; import org.nuiton.topia.TopiaContext; import org.nuiton.topia.TopiaException; @@ -482,6 +456,10 @@ return users; } + /** + *@deprecated FIXME-fdesbois-2010-12-23 : move this method into WaoUserDAO + */ + @Deprecated protected List<WaoUser> getUsersByCompany(TopiaContext transaction, Company company) throws TopiaException { WaoUserDAO dao = WaoDAOHelper.getWaoUserDAO(transaction); Modified: trunk/wao-business/src/test/java/fr/ifremer/wao/AbstractServiceTest.java =================================================================== --- trunk/wao-business/src/test/java/fr/ifremer/wao/AbstractServiceTest.java 2010-12-23 17:15:46 UTC (rev 855) +++ trunk/wao-business/src/test/java/fr/ifremer/wao/AbstractServiceTest.java 2010-12-23 19:09:52 UTC (rev 856) @@ -23,24 +23,8 @@ import fr.ifremer.wao.bean.ContactState; import fr.ifremer.wao.bean.UserRole; -import fr.ifremer.wao.entity.Boat; -import fr.ifremer.wao.entity.BoatDAO; -import fr.ifremer.wao.entity.Company; -import fr.ifremer.wao.entity.CompanyDAO; -import fr.ifremer.wao.entity.Contact; -import fr.ifremer.wao.entity.ContactDAO; -import fr.ifremer.wao.entity.ContactImpl; -import fr.ifremer.wao.entity.ElligibleBoat; -import fr.ifremer.wao.entity.ElligibleBoatDAO; -import fr.ifremer.wao.entity.SampleRow; -import fr.ifremer.wao.entity.SampleRowDAO; -import fr.ifremer.wao.entity.WaoUser; -import fr.ifremer.wao.entity.WaoUserDAO; -import fr.ifremer.wao.service.ServiceBoat; -import fr.ifremer.wao.service.ServiceContact; -import fr.ifremer.wao.service.ServiceReferential; -import fr.ifremer.wao.service.ServiceSampling; -import fr.ifremer.wao.service.ServiceSamplingImpl; +import fr.ifremer.wao.entity.*; +import fr.ifremer.wao.service.*; import org.junit.After; import org.junit.Before; import org.junit.BeforeClass; @@ -103,8 +87,7 @@ log.debug("Start manager"); } manager.start(); - serviceSampling = (ServiceSamplingImpl)manager.getServiceSampling(); - //initialize(); + serviceSampling = (ServiceSamplingImpl) manager.getServiceSampling(); } @After @@ -113,12 +96,6 @@ } /** - * Initialize data before each test. By default do nothing. - */ -// protected void initialize() { -// } - - /** * Create a company : <br /> * <ul> * <li>first name : Homer</li> Modified: trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceUserImplTest.java =================================================================== --- trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceUserImplTest.java 2010-12-23 17:15:46 UTC (rev 855) +++ trunk/wao-business/src/test/java/fr/ifremer/wao/service/ServiceUserImplTest.java 2010-12-23 19:09:52 UTC (rev 856) @@ -24,93 +24,54 @@ package fr.ifremer.wao.service; +import fr.ifremer.wao.AbstractServiceTest; import fr.ifremer.wao.TestManager; import fr.ifremer.wao.WaoBusinessException; import fr.ifremer.wao.WaoDAOHelper; import fr.ifremer.wao.bean.ObsProgram; import fr.ifremer.wao.bean.UserRole; -import fr.ifremer.wao.entity.Company; -import fr.ifremer.wao.entity.CompanyImpl; -import fr.ifremer.wao.entity.UserProfile; -import fr.ifremer.wao.entity.UserProfileDAO; -import fr.ifremer.wao.entity.UserProfileImpl; -import fr.ifremer.wao.entity.WaoUser; -import fr.ifremer.wao.entity.WaoUserImpl; -import org.junit.After; -import org.junit.AfterClass; +import fr.ifremer.wao.entity.*; import org.junit.Assert; import org.junit.Before; -import org.junit.BeforeClass; import org.junit.Test; import org.nuiton.topia.TopiaContext; +import org.nuiton.topia.TopiaException; import org.slf4j.Logger; import org.slf4j.LoggerFactory; +import java.util.ArrayList; import java.util.LinkedList; import java.util.List; /** - * * @author fdesbois <fdesbois at codelutin.com> + * @version $Id$ */ -public class ServiceUserImplTest { +public class ServiceUserImplTest extends AbstractServiceTest { - private static TestManager manager; - private static final Logger log = LoggerFactory.getLogger(ServiceUserImplTest.class); private ServiceUser service; - public ServiceUserImplTest() { - } - - @BeforeClass - public static void setUpClass() throws Exception { - manager = new TestManager(); - } - - @AfterClass - public static void tearDownClass() throws Exception { - } - @Before public void setUp() throws Exception { - manager.start(); + super.setUp(); service = manager.getServiceUser(); } - @After - public void tearDown() throws Exception { - manager.stop(); - } - /** - * Test of connect method, of class ServiceUserImpl. - */ - //@Test - public void testConnect() throws Exception { - System.out.println("connect"); - } - - /** - * Test of forgetPassword method, of class ServiceUserImpl. - */ - //@Test - public void testForgetPassword() throws Exception { - System.out.println("forgetPassword"); - } - - /** * Test of createUpdateUser method, of class ServiceUserImpl. * * This test is not complete : it doesn't check if given data are well * registered. For profile updating, it only checks that no exception is * raised. + * + * @throws Exception all exceptions */ @Test public void testCreateUpdateUser() throws Exception { - System.out.println("createUpdateUser"); + log.info("############### START ############### testCreateUpdateUser"); // create a user with no role WaoUser user = new WaoUserImpl(); @@ -122,8 +83,16 @@ List<UserProfile> profiles = new LinkedList<UserProfile>(); user.setUserProfile(profiles); - UserProfile adminProfile = new UserProfileImpl(ObsProgram.OBSMER, UserRole.ADMIN, true); - UserProfile coordinatorProfile = new UserProfileImpl(ObsProgram.OBSMER, UserRole.COORDINATOR, true); + UserProfile adminProfile = new UserProfileImpl( + ObsProgram.OBSMER, + UserRole.ADMIN, + true + ); + UserProfile coordinatorProfile = new UserProfileImpl( + ObsProgram.OBSMER, + UserRole.COORDINATOR, + true + ); profiles.add(adminProfile); @@ -149,46 +118,85 @@ service.createUpdateUser(user, false); Assert.fail("an exception should have been raised"); } catch (WaoBusinessException e) { - Assert.assertEquals(WaoBusinessException.Type.ALREADY_EXISTS, e.getType()); + Assert.assertEquals(WaoBusinessException.Type.ALREADY_EXISTS, + e.getType()); } } - /** - * Test of deleteUser method, of class ServiceUserImpl. - */ - //@Test - public void testDeleteUser() throws Exception { - System.out.println("deleteUser"); - } + @Test + public void testUpdateUser() throws WaoBusinessException, TopiaException { + log.info("############### START ############### testUpdateUser"); - /** - * Test of createUpdateCompany method, of class ServiceUserImpl. - */ - //@Test - public void testCreateUpdateCompany() throws Exception { - System.out.println("createUpdateCompany"); - } + // ---- PREPARE DATA ---- // - /** - * Test of getCompanies method, of class ServiceUserImpl. - */ - //@Test - public void testGetCompanies() throws Exception { - System.out.println("getCompanies"); + String login = "login"; + + // First instance to create the user + WaoUser user = new WaoUserImpl(); + user.setLogin(login); + user.setActive(true); + service.createUpdateUser(user, false); + + log.debug("user in session = " + user + + " [ID=" + user.getTopiaId() + "]"); + + // Create a new instance + WaoUser userToUpdate = new WaoUserImpl(); + userToUpdate.setLogin(user.getLogin()); + userToUpdate.setTopiaId(user.getTopiaId()); + userToUpdate.setActive(false); + // Avoid NPE on UserProfile for each + userToUpdate.setUserProfile(new ArrayList<UserProfile>()); + + log.debug("user to update = " + userToUpdate); + + // ---- EXECUTE ---- // + + service.createUpdateUser(userToUpdate, false); + + // ---- CHECK DATA ---- // + + WaoUser userFound = findUserByLogin(login); + + log.debug("user found = " + userFound); + + Assert.assertFalse(userFound.isActive()); } /** * Test of getObservers method, of class ServiceUserImpl. + * + * @throws Exception all exceptions */ @Test public void testGetObservers() throws Exception { - log.info("## START ## getObservers"); + log.info("############### START ############### testGetObservers"); - UserProfile adminProfile = new UserProfileImpl(ObsProgram.OBSMER, UserRole.ADMIN, true); - UserProfile coordinatorProfile = new UserProfileImpl(ObsProgram.OBSMER, UserRole.COORDINATOR, true); - UserProfile coordinatorProfile2 = new UserProfileImpl(ObsProgram.OBSMER, UserRole.COORDINATOR, true); - UserProfile observerProfile = new UserProfileImpl(ObsProgram.OBSMER, UserRole.OBSERVER, true); - UserProfile observerProfile2 = new UserProfileImpl(ObsProgram.OBSMER, UserRole.OBSERVER, true); + UserProfile adminProfile = new UserProfileImpl( + ObsProgram.OBSMER, + UserRole.ADMIN, + true + ); + UserProfile coordinatorProfile = new UserProfileImpl( + ObsProgram.OBSMER, + UserRole.COORDINATOR, + true + ); + UserProfile coordinatorProfile2 = new UserProfileImpl( + ObsProgram.OBSMER, + UserRole.COORDINATOR, + true + ); + UserProfile observerProfile = new UserProfileImpl( + ObsProgram.OBSMER, + UserRole.OBSERVER, + true + ); + UserProfile observerProfile2 = new UserProfileImpl( + ObsProgram.OBSMER, + UserRole.OBSERVER, + true + ); TopiaContext context = manager.getContext().beginTransaction(); UserProfileDAO userProfileDAO = WaoDAOHelper.getUserProfileDAO(context); @@ -266,10 +274,12 @@ /** * Test of initApplication method, of class WaoGlobal. + * + * @throws Exception all exceptions */ @Test public void testCreateDefaultAdmin() throws Exception { - log.info("## START ## createDefaultAdmin"); + log.info("############### START ############### testCreateDefaultAdmin"); service.createDefaultAdmin();

1 0

[Suiviobsmer-commits] r855 - trunk/wao-ui/src/main/webapp
by bleny＠users.labs.libre-entreprise.org Dec. 23, 2010

Dec. 23, 2010

Author: bleny Date: 2010-12-23 17:15:46 +0000 (Thu, 23 Dec 2010) New Revision: 855 Log: disable activity calender in 'boats' page Modified: trunk/wao-ui/src/main/webapp/Boats.tml Modified: trunk/wao-ui/src/main/webapp/Boats.tml =================================================================== --- trunk/wao-ui/src/main/webapp/Boats.tml 2010-12-23 15:45:24 UTC (rev 854) +++ trunk/wao-ui/src/main/webapp/Boats.tml 2010-12-23 17:15:46 UTC (rev 855) @@ -185,6 +185,7 @@ <t:delegate to="activeBoatInfosBlock" /> </t:if>  +   <t:unless t:test="user.admin"> <div id="so-boats-boat-infos-message">

1 0

[Suiviobsmer-commits] r854 - trunk/wao-ui/src/main/webapp
by bleny＠users.labs.libre-entreprise.org Dec. 23, 2010

Dec. 23, 2010

Author: bleny Date: 2010-12-23 15:45:24 +0000 (Thu, 23 Dec 2010) New Revision: 854 Log: fix XML Modified: trunk/wao-ui/src/main/webapp/ContactForm.tml Modified: trunk/wao-ui/src/main/webapp/ContactForm.tml =================================================================== --- trunk/wao-ui/src/main/webapp/ContactForm.tml 2010-12-23 15:08:24 UTC (rev 853) +++ trunk/wao-ui/src/main/webapp/ContactForm.tml 2010-12-23 15:45:24 UTC (rev 854) @@ -114,7 +114,7 @@ <div class="t-beaneditor-row"> <t:label t:for="nbObservants" /> ${contact.nbObservants} - </div + </div> <div class="t-beaneditor-row"> <t:label t:for="beginDate" /> <t:output value="contact.observationBeginDate" format="dateTimeFormat" />

1 0