camp prototype model #1031

pmattmann · 2021-02-28T13:06:23Z

Isseu: #834

backend
frontend

pmattmann · 2021-02-28T21:33:12Z

Ich habe es wie folgt implementiert:

Category und Activity leiten von AbstractContentNodeOwner ab.
AbstractContentNodeOwner zeigt auf das Root-Element vom ContentNode-Tree.
Jeder ContentNode zeigt auf dessen Owner (AbstractContentNodeOwner).
Jeder ContentNode zeigt auf dessen Parent ContentNode. Ausnahme ist das Root-Element, dieses hat keinen weiteren Parent.

Die Zeiger von ContentNode auf AbstractContentNodeOwner benötige ich, um das Interface BelongsToCamp implementieren zu können. Sprich, damit ich herausfinden kann, zu welchem Camp ein ContentNode gehört.

Mögliche Alternative:
Anstelle vom Zeiger ContentNode -> AbstractContentNodeOwner könnte man auch folgendes machen:
OneToOne-Beziehung zwischen AbstractContentNodeOwner <-> ContentNode kombiniert mit einem Root-ContentNode-Zeiger auf allen ContentNodes ContentNode -> Root-ContentNode.

edit

manuelmeister · 2021-03-03T08:11:53Z

Für was habt ihr euch entschieden?

carlobeltrame · 2021-03-03T10:59:08Z

Für die zweite Option, da dann der ContentNode-Baum unabhängiger vom jeweiligen Owner ist. Er ist immer noch nicht ganz unabhängig, da es für den Berechtigungs-Check nötig ist, vom ContentNode aus zum Camp zu finden. Also wird (noch) nix mit komplett unabhängigen, wiederverwendbaren ContentNode-Bäumen.

Wir haben noch kurz eine dritte, noch "extremere", noch stärker normalisierte Variante besprochen, nämlich gar keinen root Zeiger zu haben. Um von einem beliebigen ContentNode zur Activity zu kommen müsste man sich dann durch die Baum-Hierarchie (parent vom parent vom parent ...) nach oben hangeln, bis man an einem Root Node ankommt. Das gäbe zwar am wenigsten Duplikation (und somit am wenigsten Potenzial für inkonsistente Daten beim vielen Klonen das wir machen werden), aber wäre mühsamer auszuführen, da Doctrine dazu wohl bei einer Baumtiefe n auch n Datenbank-Queries absetzen würde. Und da dieser Permissions-Check voraussichtlich relativ oft vorkommt, haben wir uns für den Moment dagegen entschieden.

Ich bin der Meinung dass die gewählte Option im Moment ausreicht, und trotzdem noch flexibel zu migrieren ist, falls wir uns umentscheiden sollten.

use ContentTypeStrategy in service

…e-model

pmattmann · 2021-03-04T22:16:51Z

PR is ready to merge.

Enthalten:

Umbau backend
Anpassung frontend, so dass bestehender Code mit Backend funktioniert

Nicht enthalten:

Frontend:
- Category-Verwaltung
  - Bevorzugte ContentTypes wählen
  - ContentNode-Struktur erstellen
- Activity
  - ContentNode-Struktur erstellen

Bevorzugte ContentTypes pro Category ist ein neues Feature -> eigener PR.
ContentNode-Struktur erstellen lässt sich erst implementierne, wenn wir den "SpaltenLayout-ContentType" haben.

Temporärer Effekt:
Jeder Block kann genau ein ContentNode haben (der RootNode).

carlobeltrame

Nice work! Ein paar Anmerkungen (wie immer), aber hoffentlich keine fundamentalen Änderungen mehr diesmal ;)

backend/module/eCampApi/src/eCampApi/V1/Rpc/Index/IndexController.php

backend/module/eCampCore/data/prod/CampPrototypeData.php

backend/module/eCampCore/data/prod/MaterialListPrototypeData.php

backend/module/eCampCore/src/ContentType/BaseContentTypeService.php

carlobeltrame · 2021-03-05T13:16:06Z

backend/module/eCampCore/src/ContentType/BaseContentTypeService.php

-        if (is_subclass_of($this->entityClass, BaseContentTypeEntity::class)) {
-            $q->join('row.contentNode', 'cn');
-            $q->join('cn.activity', 'a');
-            $q->andWhere($this->createFilter($q, Camp::class, 'a', 'camp'));
-        }


Woher hat dieser Filter vormals gewusst, um welche Activity es gerade geht? Wo ist/war der Filter definiert?

ContentNode hat auf Activity gezeigt.
Activity auf Camp.
Camp-Filter schränkt die Sicht von User ein.

Durch diese Joins wurden die möglichen sichtbaren ContentTypes eingeschränkt.

backend/module/eCampCore/src/Hydrator/ContentNodeHydrator.php

backend/module/eCampCore/test/Entity/CategoryTest.php

backend/module/eCampCore/test/Entity/ContentNodeTest.php

backend/module/eCampCore/test/Service/CampServiceTest.php

backend/module/eCampLib/src/Acl/AclAssertion.php

backend/module/eCampCore/src/EntityService/ContentNodeService.php

carlobeltrame

Super, danke! Ich mach mich dannmal dran, dass man wieder mehrere ContentNodes in einer Activity haben kann :)

BacLuc · 2021-03-11T20:52:52Z

Super, jetzt können sich auch andere einen Teil des Kuchens schnappen und loslegen.

Habe es getestet, was nicht funktioniert:

Man kann nur einen ContentNode (Story, MaterialList) auf einer Activity haben.
Wenn man einen neuen hinzufügt, wird der alte gelöscht.
Bei MaterialList kann man für die Activity keine Items hinzufügen
print geht kaputt

Das sollte man später noch fixen.

Die Logs vom Print:

ecamp3-print              |  ERROR  Error in fetch(): this.activity.activityCategory is not a function
ecamp3-print              | 
ecamp3-print              |   at VueComponent.fetch (components/program-schedule-entry.js:131:49)
ecamp3-print              |   at processTicksAndRejections (internal/process/task_queues.js:93:5)
ecamp3-print              |   at async VueComponent.serverPrefetch (server.js:1461:5)
ecamp3-print              |   at async Promise.all (index 0)
ecamp3-print              | 
ecamp3-print              | 
ecamp3-print              |  ERROR  [Vue warn]: Error in render: "TypeError: Cannot read property 'name' of null"
ecamp3-print              | 
ecamp3-print              | found in
ecamp3-print              | 
ecamp3-print              | ---> <ProgramScheduleEntry> at components/ProgramScheduleEntry.vue
ecamp3-print              |        <ProgramDay> at components/ProgramDay.vue
ecamp3-print              |          <ProgramPeriod> at components/ProgramPeriod.vue
ecamp3-print              |            <Program> at components/Program.vue
ecamp3-print              |              <Pages/index.vue> at pages/index.vue
ecamp3-print              |                <Nuxt>
ecamp3-print              |                  <VMain>
ecamp3-print              |                    <VApp>
ecamp3-print              |                      <Layouts/default.vue> at layouts/default.vue
ecamp3-print              |                        <Root>
ecamp3-print              | 
ecamp3-print              | 
ecamp3-print              |  ERROR  Error in fetch(): this.activity.activityCategory is not a function
ecamp3-print              | 
ecamp3-print              |   at VueComponent.fetch (components/program-schedule-entry.js:131:49)
ecamp3-print              |   at processTicksAndRejections (internal/process/task_queues.js:93:5)
ecamp3-print              |   at async VueComponent.serverPrefetch (server.js:1461:5)
ecamp3-print              |   at async Promise.all (index 0)
ecamp3-print              | 
ecamp3-print              | 
ecamp3-print              |  ERROR  [Vue warn]: Error in render: "TypeError: Cannot read property 'name' of null"
ecamp3-print              | 
ecamp3-print              | found in
ecamp3-print              | 
ecamp3-print              | ---> <ProgramScheduleEntry> at components/ProgramScheduleEntry.vue
ecamp3-print              |        <ProgramDay> at components/ProgramDay.vue
ecamp3-print              |          <ProgramPeriod> at components/ProgramPeriod.vue
ecamp3-print              |            <Program> at components/Program.vue
ecamp3-print              |              <Pages/index.vue> at pages/index.vue
ecamp3-print              |                <Nuxt>
ecamp3-print              |                  <VMain>
ecamp3-print              |                    <VApp>
ecamp3-print              |                      <Layouts/default.vue> at layouts/default.vue
ecamp3-print              |                        <Root>

…e-model

pmattmann · 2021-03-13T20:20:08Z

Man kann nur einen ContentNode (Story, MaterialList) auf einer Activity haben.
Wenn man einen neuen hinzufügt, wird der alte gelöscht.

Bei MaterialList kann man für die Activity keine Items hinzufügen

print geht kaputt

MaterialList & Print habe ich geflickt.
Mehrere ContentNode geht erst wieder, wenn wir den ContentType "Column" haben.
-> das ist "absichtlich" kaputt ;)

pmattmann added 2 commits February 28, 2021 14:03

camp-prototype-model backend

a21828f

more unittests

7a14382

pmattmann self-assigned this Feb 28, 2021

pmattmann added the type: Backend label Feb 28, 2021

pmattmann added this to the MVP (v3.0) milestone Feb 28, 2021

ContentNodeHydrator fix and unittest

825b465

pmattmann added the Meeting Discuss Am nächsten Core-Meeting besprechen label Feb 28, 2021

ContentNode as closed tree

098be89

pmattmann removed the Meeting Discuss Am nächsten Core-Meeting besprechen label Mar 2, 2021

pmattmann added 6 commits March 3, 2021 17:25

root is never null

3cfa32e

remove ContentTypeStrategyProviderInjector;

fcbee7e

use ContentTypeStrategy in service

additional unittests

fdaed1f

fix activity-view

a190705

Merge remote-tracking branch 'ecamp/devel' into feature/camp-prototyp…

716809e

…e-model

add xdebug explicitly

10b9940

carlobeltrame requested changes Mar 5, 2021

View reviewed changes

pmattmann added 3 commits March 5, 2021 20:47

fix camp-creation

0089b45

code review fixes

ce21e0c

fix MaterialListPrototypeData

79ba9cc

pmattmann force-pushed the feature/camp-prototype-model branch 2 times, most recently from fa7500a to c176938 Compare March 6, 2021 11:18

pmattmann added 4 commits March 6, 2021 12:18

fix renaming

c176938

implement ContentNode jsonConfig

dd8267d

rename contentNode collections

8042d0d

category rootContentNode test

1ddd102

pmattmann requested a review from carlobeltrame March 6, 2021 15:15

carlobeltrame reviewed Mar 6, 2021

View reviewed changes

backend/module/eCampCore/src/EntityService/ContentNodeService.php Show resolved Hide resolved

add ACL to ContentNodeService

a105fd0

carlobeltrame approved these changes Mar 7, 2021

View reviewed changes

BacLuc approved these changes Mar 11, 2021

View reviewed changes

pmattmann added 3 commits March 13, 2021 21:01

Merge remote-tracking branch 'ecamp/devel' into feature/camp-prototyp…

470d2a7

…e-model

fix materiallist

c7e7080

fix printing

e485be9

pmattmann merged commit bccd5cc into ecamp:devel Mar 13, 2021

pmattmann deleted the feature/camp-prototype-model branch March 13, 2021 20:24

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

camp prototype model #1031

camp prototype model #1031

Uh oh!

pmattmann commented Feb 28, 2021 •

edited

Loading

Uh oh!

pmattmann commented Feb 28, 2021

Uh oh!

manuelmeister commented Mar 3, 2021

Uh oh!

carlobeltrame commented Mar 3, 2021

Uh oh!

pmattmann commented Mar 4, 2021

Uh oh!

carlobeltrame left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

carlobeltrame Mar 5, 2021

Uh oh!

pmattmann Mar 5, 2021

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

carlobeltrame left a comment

Uh oh!

BacLuc commented Mar 11, 2021 •

edited by pmattmann

Loading

Uh oh!

pmattmann commented Mar 13, 2021

Uh oh!

Uh oh!

camp prototype model #1031

camp prototype model #1031

Uh oh!

Conversation

pmattmann commented Feb 28, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pmattmann commented Feb 28, 2021

Uh oh!

manuelmeister commented Mar 3, 2021

Uh oh!

carlobeltrame commented Mar 3, 2021

Uh oh!

pmattmann commented Mar 4, 2021

Uh oh!

carlobeltrame left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

carlobeltrame Mar 5, 2021

Choose a reason for hiding this comment

Uh oh!

pmattmann Mar 5, 2021

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

carlobeltrame left a comment

Choose a reason for hiding this comment

Uh oh!

BacLuc commented Mar 11, 2021 • edited by pmattmann Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pmattmann commented Mar 13, 2021

Uh oh!

Uh oh!

pmattmann commented Feb 28, 2021 •

edited

Loading

BacLuc commented Mar 11, 2021 •

edited by pmattmann

Loading