Feature/publication state #274

PhilippeFerreiraDeSousa · 2017-10-20T17:17:45Z

Draft : Seul les membres du club ont accès aux brouillons dans la liste des publications du club.
Scheduled : Cette publication apparaîtra seulement sur le calendrier uPont.
Published : Cette publication sera publique sur uPont, vous pourrez envoyer un mail plus tard.
Emailed: La publication sera publiée et envoyée par mail à tous les utilisateurs de uPont qui suivent le club.

… de création d'event et news

…tate

…appear

…ublications

…overlapping events at the creation or modification of an event. Check for authorization on drafts on /events and /newsitems, create all personnel messages as 'Published'

…rouillons d'autrui

LouisTrezzini

Je n'ai pas eu le courage de tout relire en détail ni de tester en local

LouisTrezzini · 2017-10-21T21:33:45Z

back/app/DoctrineMigrations/Version20171015200834.php

+        // this up() migration is auto-generated, please modify it to your needs
+        $this->abortIf($this->connection->getDatabasePlatform()->getName() !== 'mysql', 'Migration can only be executed safely on \'mysql\'.');
+
+        $this->addSql('ALTER TABLE Post ADD publicationState VARCHAR(255) DEFAULT \'Published\', DROP send_mail');


Est-ce qu'on a pas envie d'attribuer le bon publishedState suivant la valeur actuelle de send_mail lors de la migration ?

Oui oui, faut faire des boucles php ici avec des addSql partout ? o:)

LouisTrezzini · 2017-10-21T21:39:15Z

back/src/KI/CoreBundle/Controller/ResourceController.php

@@ -12,12 +12,12 @@ class ResourceController extends LikeableController
     * @param  boolean $auth Un override éventuel pour le check des permissions
     * @return Response
     */
-    public function getAll($auth = false)
+    public function getAll($auth = false, array $findBy = [])


Pourquoi un $findBy sur le getAll des resources ?

Je l'utilise pour requêter les messages à /newsitems?name=message

if ($request->query->get('name') == 'message') { $findBy = array('name' => 'message'); return $this->getAll($this->is('EXTERIEUR'), $findBy); }

En soi donner "message" comme titre de news aux messages de l'accueil est très douteux

LouisTrezzini · 2017-10-21T21:43:15Z

back/src/KI/CoreBundle/Helper/PaginateHelper.php

@@ -33,11 +33,6 @@ public function paginateData(EntityRepository $repository, array $findBy = [])
        $queryBuilder = $repository->createQueryBuilder('o');
        $request = $this->request->query;

-        // On s'assure de bien recevoir des arrays
-        foreach ($findBy as $key => $value) {
-            $findBy[$key] = array($value);


À garder avec un truc du genre $aRecords = is_array($aRecords) ? $aRecords : array($aRecords);

LouisTrezzini · 2017-10-21T21:50:23Z

back/src/KI/PublicationBundle/Controller/EventsController.php

@@ -63,6 +69,9 @@ public function getEventsAction()
    public function getEventAction($slug)
    {
        $event = $this->getOne($slug);
+        if ($event->getPublicationState() == 'Draft' && !$this->isClubMember($event->getAuthorClub())) {
+            return $this->json('Tu n\'es pas autorisé à lire ce brouillon !', 403);


throw $this->createAccessDeniedException('You cannot access this page!');

LouisTrezzini · 2017-10-21T21:51:24Z

back/src/KI/PublicationBundle/Controller/EventsController.php

+            $this->getUser()->getId(),
+            $request->query->get('publicationState'),
+            $request->query->get('limit'),
+            $request->query->get('page'));


Problème d'indentation

Regarde bien, les lignes doublements indentées sont en argument de getAllowedEvents

LouisTrezzini · 2017-10-21T22:26:58Z

front/app/js/controllers/users/publications/list.html

            ng-repeat="publication in events.data | concat: newsItems.data | orderBy: 'date' | reverse"
-            ng-if="publication.entry_method != 'Ferie'">
+            ng-if="publication.entry_method != 'Ferie' && ($root.hasClub(publication.author_club.slug) || ['Published', 'Emailed'].includes(publication.publication_state))">
+            <up-ribbon ng-repeat="(state, item) in pub_info" class="Ribbon--{{ pub_info[publication.publication_state].color }}" ng-hide="editSlug == publication.slug"


pourquoi tu boucles alors que tu sais que publication.publication_state == state ?

Il y a 4 ribbons différents, un membre voit les publications de son club pour les 4 états.
On est dans list.html à l'intérieur d'un ng-repeat="publication in events.data | concat: newsItems.data | orderBy: 'date' | reverse" donc publication.publication_state change dans ce feed de publications

LouisTrezzini · 2017-10-21T22:27:58Z

front/app/js/controllers/users/publications/list.html

            ng-repeat="publication in events.data | concat: newsItems.data | orderBy: 'date' | reverse"
-            ng-if="publication.entry_method != 'Ferie'">
+            ng-if="publication.entry_method != 'Ferie' && ($root.hasClub(publication.author_club.slug) || ['Published', 'Emailed'].includes(publication.publication_state))">


pourquoi tu ne renvoies pas que les bonnes publications aux bons users ? pourquoi il y a de la logique dans le front ?

On veut juste pas afficher dans le feed les événements planifiés des autres assos mais y laisser accès, vu qu'on utlise la même route ownClub pour le calendrier et qu'on peut l'utiliser pour autre chose plus tard.
On peut simplifier en $root.hasClub(publication.author_club.slug) || publication.publication_state != 'scheduled'

LouisTrezzini · 2017-10-21T22:30:59Z

front/app/js/controllers/users/publications/list.html

            </div>
            <div up-likes show="false" objet="publication"
-                 url="(publication.start_date ? 'events/' : 'newsitems/') + publication.slug"></div>
+                 url="(publication.start_date ? 'events/' : 'newsitems/') + publication.slug"
+                 ng-if="!['Draft', 'Scheduled'].includes(publication.publication_state)"></div>


LouisTrezzini · 2017-10-21T22:33:02Z

front/app/js/controllers/users/publications/post.html

+                <select class="Input" ng-model="post.entry_method">
+                    <option value="Libre" selected ng-disabled="modify && initialEntryMethod == 'Shotgun'">Entrée libre</option>
+                    <option value="Shotgun">Shotgun</option>
+                    <option value="Ferie" ng-if="$root.isAdmin" ng-disabled="modify && initialEntryMethod == 'Shotgun'">Ferie</option>


ng-if="$root.isAdmin"

...
je pense que tu peux supprimer tout référence à "Ferie" du coup

C'est pour faire apparaître les vacances scolaires et jours fériés sur le calendrier. Ca aurait plus de sens d'être sur le panneau d'administration mais pour l'instant ça fait de mal à personne ici. On pourra statuer là-dessus plus tard je pense

LouisTrezzini · 2017-10-21T22:34:44Z

front/app/js/controllers/users/publications/post.html

@@ -82,9 +95,10 @@
                    <div class="InputButton" ng-if="!modify" ng-class="{'active': postFiles.length}">
                        <i class="fa fa-file-o"></i>
                        <span ng-repeat="file in postFiles"> {{ file.name ? file.name : 'Fichier' }} </span>
-                        <button class="InputButton__fileinput" ngf-select="selectFiles($files)" multiple></button>
+                        <button class="InputButton__fileinput" ng-select="selectFiles($files)" multiple></button>


ngf-select pour les fichiers...

PhilippeFerreiraDeSousa

Je vais revoir tout ça

PhilippeFerreiraDeSousa · 2017-10-22T14:45:46Z

back/app/DoctrineMigrations/Version20171015200834.php

+        // this up() migration is auto-generated, please modify it to your needs
+        $this->abortIf($this->connection->getDatabasePlatform()->getName() !== 'mysql', 'Migration can only be executed safely on \'mysql\'.');
+
+        $this->addSql('ALTER TABLE Post ADD publicationState VARCHAR(255) DEFAULT \'Published\', DROP send_mail');


Oui oui, faut faire des boucles php ici avec des addSql partout ? o:)

PhilippeFerreiraDeSousa · 2017-10-22T14:50:27Z

back/src/KI/CoreBundle/Controller/ResourceController.php

@@ -12,12 +12,12 @@ class ResourceController extends LikeableController
     * @param  boolean $auth Un override éventuel pour le check des permissions
     * @return Response
     */
-    public function getAll($auth = false)
+    public function getAll($auth = false, array $findBy = [])


Je l'utilise pour requêter les messages à /newsitems?name=message

if ($request->query->get('name') == 'message') { $findBy = array('name' => 'message'); return $this->getAll($this->is('EXTERIEUR'), $findBy); }

En soi donner "message" comme titre de news aux messages de l'accueil est très douteux

PhilippeFerreiraDeSousa · 2017-10-22T14:52:44Z

back/src/KI/PublicationBundle/Controller/EventsController.php

+            $this->getUser()->getId(),
+            $request->query->get('publicationState'),
+            $request->query->get('limit'),
+            $request->query->get('page'));


Regarde bien, les lignes doublements indentées sont en argument de getAllowedEvents

PhilippeFerreiraDeSousa · 2017-10-22T14:55:44Z

back/src/KI/PublicationBundle/Controller/EventsController.php

@@ -63,6 +69,9 @@ public function getEventsAction()
    public function getEventAction($slug)
    {
        $event = $this->getOne($slug);
+        if ($event->getPublicationState() == 'Draft' && !$this->isClubMember($event->getAuthorClub())) {


Oui, je vais les mettre en slug. C'était comme ça à la base par cohérence avec l'entryMethod des Events 'Shotgun', 'Libre', 'Ferie' mais c'est plus très justifié maintenant

Ce qui est published est sur le newsfeed et le calendrier, ce qui est scheduled n'est que sur le newsfeed des membre de l'asso qui peuvent l'éditer et sur le calendrier (mais accessible par recherche dans la topbar et par requête au back)

PhilippeFerreiraDeSousa · 2017-10-22T15:18:15Z

back/src/KI/PublicationBundle/Controller/EventsController.php

+        $endDate = $request->query->get('endDate');
+        $events = $this->repository->findBy(array('publicationState' => array('Scheduled', 'Published', 'Emailed')));
+        $matchedEvents = array();
+        foreach ($events as $event) {


Oui, il faut mettre cette logique dans une requête DQL dans le repository

PhilippeFerreiraDeSousa · 2017-10-22T16:01:14Z

front/app/js/controllers/users/calendar.js

@@ -54,7 +54,7 @@ angular.module('upont')
                },
                resolve: {
                    events: ['$resource', function($resource) {
-                        return $resource(apiPrefix + 'own/events').query().$promise;
+                        return $resource(apiPrefix + 'own/events?publicationState=Scheduled,Published,Emailed').query().$promise;


Les brouillons sont privés, pas sur le calendrier

PhilippeFerreiraDeSousa · 2017-10-22T16:09:52Z

front/app/js/controllers/users/publications/list.html

            ng-repeat="publication in events.data | concat: newsItems.data | orderBy: 'date' | reverse"
-            ng-if="publication.entry_method != 'Ferie'">
+            ng-if="publication.entry_method != 'Ferie' && ($root.hasClub(publication.author_club.slug) || ['Published', 'Emailed'].includes(publication.publication_state))">


On veut juste pas afficher dans le feed les événements planifiés des autres assos mais y laisser accès, vu qu'on utlise la même route ownClub pour le calendrier et qu'on peut l'utiliser pour autre chose plus tard.
On peut simplifier en $root.hasClub(publication.author_club.slug) || publication.publication_state != 'scheduled'

PhilippeFerreiraDeSousa · 2017-10-22T16:15:43Z

front/app/js/controllers/users/publications/list.html

            ng-repeat="publication in events.data | concat: newsItems.data | orderBy: 'date' | reverse"
-            ng-if="publication.entry_method != 'Ferie'">
+            ng-if="publication.entry_method != 'Ferie' && ($root.hasClub(publication.author_club.slug) || ['Published', 'Emailed'].includes(publication.publication_state))">
+            <up-ribbon ng-repeat="(state, item) in pub_info" class="Ribbon--{{ pub_info[publication.publication_state].color }}" ng-hide="editSlug == publication.slug"


Il y a 4 ribbons différents, un membre voit les publications de son club pour les 4 états.
On est dans list.html à l'intérieur d'un ng-repeat="publication in events.data | concat: newsItems.data | orderBy: 'date' | reverse" donc publication.publication_state change dans ce feed de publications

PhilippeFerreiraDeSousa · 2017-10-22T16:16:40Z

front/app/js/controllers/users/publications/list.html

            </div>
            <div up-likes show="false" objet="publication"
-                 url="(publication.start_date ? 'events/' : 'newsitems/') + publication.slug"></div>
+                 url="(publication.start_date ? 'events/' : 'newsitems/') + publication.slug"
+                 ng-if="!['Draft', 'Scheduled'].includes(publication.publication_state)"></div>


PhilippeFerreiraDeSousa · 2017-10-22T16:23:15Z

front/app/js/controllers/users/publications/post.html

+                <select class="Input" ng-model="post.entry_method">
+                    <option value="Libre" selected ng-disabled="modify && initialEntryMethod == 'Shotgun'">Entrée libre</option>
+                    <option value="Shotgun">Shotgun</option>
+                    <option value="Ferie" ng-if="$root.isAdmin" ng-disabled="modify && initialEntryMethod == 'Shotgun'">Ferie</option>


C'est pour faire apparaître les vacances scolaires et jours fériés sur le calendrier. Ca aurait plus de sens d'être sur le panneau d'administration mais pour l'instant ça fait de mal à personne ici. On pourra statuer là-dessus plus tard je pense

PhilippeFerreiraDeSousa · 2017-10-22T17:01:03Z

Editer la publucation pour changer l'état dans la dropdown c'était la solution rapide. Je vais réfléchir au système incrémental qui est plus élaboré mais faut que je trouve du temps

…Action selection logic into repo

… a row and request to checkDates after changing slug To do : test data migration, correct get to own pubilcations, unpredictable redactor. The calendar library should be changed in the future. Some fonts are missing for some months, event should inherit from newsitem and why the fuck are personnel messages newsitems with title 'message'

…n pagination

PhilippeFerreiraDeSousa added 14 commits October 14, 2017 18:19

Add states Draft, Scheduled, Published to events only first to test

26d79b6

Déplacement de l'état de publication sur Post. Arrangement formulaire…

57dcede

… de création d'event et news

Creation and modification of events work, cannot modify to inferior s…

8be81e6

…tate

Publications with ribbon, draft events from other club members don't …

adf716a

…appear

La modification des events/news se fait directement à l'endroit des p…

dbcd096

…ublications

add cancelation of publication edit

4d40da7

remove alertify ask for mail

d3187ad

Add ClubUser fixture, remove deprecated doctrine assertion constraint

ab5969a

Change publication ticket color to publication state color

0e72392

Fix multiple pagination, add /events/check-dates/{slug} to check for …

e81e438

…overlapping events at the creation or modification of an event. Check for authorization on drafts on /events and /newsitems, create all personnel messages as 'Published'

Small front factorization

ca0a65a

Back propre sur les requêtes d'events/newsitems qui ne sont pas des b…

4a70526

…rouillons d'autrui

Adaptation aux users de l'administration

e0928be

Oops typos

469d085

LouisTrezzini reviewed Oct 21, 2017

View reviewed changes

PhilippeFerreiraDeSousa commented Oct 22, 2017

View reviewed changes

PhilippeFerreiraDeSousa added 5 commits October 29, 2017 22:57

change style

cc330f1

migrate data send_mail into publicationState, move getEventCheckDates…

daad1bc

…Action selection logic into repo

fix data migration down

a82e2fc

Not working version on /club/slug/newsitems but checking totalpages i…

a2ce603

…n pagination

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/publication state #274

Feature/publication state #274

PhilippeFerreiraDeSousa commented Oct 20, 2017

LouisTrezzini left a comment

LouisTrezzini Oct 21, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

LouisTrezzini Oct 21, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

LouisTrezzini Oct 21, 2017

LouisTrezzini Oct 21, 2017

LouisTrezzini Oct 21, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

LouisTrezzini Oct 21, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

LouisTrezzini Oct 21, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

LouisTrezzini Oct 21, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

LouisTrezzini Oct 21, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

LouisTrezzini Oct 21, 2017

PhilippeFerreiraDeSousa left a comment

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa Oct 22, 2017

PhilippeFerreiraDeSousa commented Oct 22, 2017

Feature/publication state #274

Are you sure you want to change the base?

Feature/publication state #274

Conversation

PhilippeFerreiraDeSousa commented Oct 20, 2017

LouisTrezzini left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

PhilippeFerreiraDeSousa left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

PhilippeFerreiraDeSousa commented Oct 22, 2017