May 2014 - Pollen-devel - mailman.cloud.codelutin.com

Quelques remarques
by Kevin Morin 20 May '14

20 May '14

Salut, voilà quelques remarques concernant le code JS dans l'ui angular : - dans pollCtrl, il y a la méthode $scope.delete. Or, delete est un mot réservé JS, ce serait bien de renommer la méthode - attention, certaines lignes sont très longues... - ça manque un peu de commentaires - c'est une question de goût, mais perso, j'aime bien qu'il y ait des espaces autours des '+' J'ai juste regardé vite fait la forme, j'ai pas du tout regardé le fond encore. Kevin

1 0

Sauvegarde automatique des données, oh non!
by Tony Chemit 12 May '14

12 May '14

Hello, J'ai remarqué qu'il y avait un timeout de sauvegarde des données, je suis pas trop ok pour ça pour les raisons suivantes : 1) ça n'arrête pas de sauver alors qu'il n'y a pas de modification, c'est moche :( 2) On sauve à chaque modification, donc pas besoin en plus de modifier tous les ... secondes Bien s'assurer qu'à la sortie des différentes éditeurs ça sauve bien, je dis ça car pour la description justement ça ne marche pas! et on doit alors attendre la sauvegarde automatique ;) Pour l'écran de settings, je suppose qu'on ne sauvegardera qu'à la sortie de la popup (ou future page?), perso la popup je l'aime pas trop, je préfèrerais un plain-page avec un menu static bootstrap sur le côté je trouve plus pratique intuitif jolie :) My 2 cents, tony. -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 http://www.codelutin.com email: chemit(a)codelutin.com twitter: https://twitter.com/tchemit

2 1

Bug Rest-api
by a.garandel＠dralagen.fr 12 May '14

12 May '14

Bonjour, Le serveur n'arrive pas a récupérer les données avec la méthode PUT ---------------------------------------------------------- Caused by: java.lang.NullPointerException at com.google.common.base.Preconditions.checkNotNull(Preconditions.java:213) at org.chorem.pollen.services.service.PollenServiceSupport.checkNotNull(PollenServiceSupport.java:232) at org.chorem.pollen.services.service.PollService.editPoll(PollService.java:159) at org.chorem.pollen.rest.api.v1.PollApi.editPoll(PollApi.java:95) ----------------------------------------------------------- Si je change dans le mapping : ----------------------------------------------------------- ligne 74: - PUT /v1/polls/{pollId} PollApi.editPoll + POST /v1/polls/{pollId} PollApi.editPoll ligne 77 - POST /v1/polls/{pollId} PollApi.clonePoll + PUT /v1/polls/{pollId} PollApi.clonePoll ----------------------------------------------------------- du coup ça marche. Mais il me demande d'envoyer les choix mais je sais que ça c'est pas important. Adrien. PS : je sais pas si je dois poster ça sur redmine pour un meilleur suivis ?

2 3

Persistance des données
by a.garandel＠dralagen.fr 12 May '14

12 May '14

Bonjour, Il semblerait que la sauvegarde en base ne marche plus sur l’édition pourtant aucune erreur lors de la requête ni sur le serveur. Voilà tout petit retour, j'en prend pas compte pour l'ui pour l'instant. Adrien

1 0

Re: [Pollen-devel] [Pollen-commits] r3915 - in trunk/pollen-ui-angular/src/main/webapp: js/controllers partials
by Tony Chemit 06 May '14

06 May '14

On Tue, 6 May 2014 10:56:15 +0200 (CEST) garandel(a)users.chorem.org wrote: > Author: garandel > Date: 2014-05-06 10:56:15 +0200 (Tue, 06 May 2014) > New Revision: 3915 > > Url: http://forge.chorem.org/projects/pollen/repository/revisions/3915 > > Log: > update settings popup, and date popup > > > Modified: > trunk/pollen-ui-angular/src/main/webapp/js/controllers/pollCtrl.js > trunk/pollen-ui-angular/src/main/webapp/partials/big-poll.html > trunk/pollen-ui-angular/src/main/webapp/partials/inline-poll.html > trunk/pollen-ui-angular/src/main/webapp/partials/poll-popupChoice.html > trunk/pollen-ui-angular/src/main/webapp/partials/poll-popupSettings.html > > Modified: trunk/pollen-ui-angular/src/main/webapp/js/controllers/pollCtrl.js > =================================================================== > --- trunk/pollen-ui-angular/src/main/webapp/js/controllers/pollCtrl.js 2014-05-05 15:39:10 UTC (rev 3914) > +++ trunk/pollen-ui-angular/src/main/webapp/js/controllers/pollCtrl.js 2014-05-06 08:56:15 UTC (rev 3915) > @@ -172,7 +172,30 @@ > > var PollPopupSettingsCtrl = function ($scope, $filter, $modalInstance, poll) { > $scope.poll = poll; > + $scope.toDay = new Date(); > > + $scope.$watch('poll.beginDate', function() { > + if (poll.beginChoiceDate) { > + if (poll.beginChoiceDate < poll.beginDate) { > + poll.beginChoiceDate = poll.beginDate; > + } > + } > + else { > + poll.beginChoiceDate = poll.beginDate; > + } > + }); > + > + $scope.$watch('poll.endDate', function() { > + if (poll.endChoiceDate) { > + if (poll.endChoiceDate > poll.endDate) { > + poll.endChoiceDate = poll.endDate; > + } > + } > + else { > + poll.endChoiceDate = poll.endDate; > + } > + }); > + Je ne pense pas qu'il faille faire ça. De manière générale, l'ui n'a pas à positionner des valeurs par défaut, c'est la couche REST à lui fournir ce dont elle a besoin. Si on n'a pas de date de fin de choix, y'a une raison, et donc ça doit rester null tant que l'utilisateur n'en a pas choisi une. Il faudrait peut-être rediscuter de tout ça de vive voix cet après-midi. tony. -- Tony Chemit -------------------- tél: +33 (0) 2 40 50 29 28 http://www.codelutin.com email: chemit(a)codelutin.com twitter: https://twitter.com/tchemit

1 0