search

OpenWebconcept / plugin-pdc-base

GNU General Public License v3.0
1 stars 4 forks source link

Fix/updater #17

Closed sanderdekroon closed 1 year ago

sanderdekroon commented 1 year ago

fixes #16

Laat even weten of dit een gewenste oplossing is. De isExtendedClass() oplossing kan ook, maar zorgt er wel voor dat we dit update systeem niet kunnen uitrollen naar child plugins.

mvdhoek1 commented 1 year ago

fixes #16

Laat even weten of dit een gewenste oplossing is. De isExtendedClass() oplossing kan ook, maar zorgt er wel voor dat we dit update systeem niet kunnen uitrollen naar child plugins.

Jawel toch? Misschien is de naamgeving niet helemaal juist maar als je in de child plugin de checkForUpdate() method toevoegt dan dan geeft het volgende een false terug:

    protected function isExtendedClass(): bool
    {
        return self::NAME !== static::NAME;
    }
sanderdekroon commented 1 year ago

My god, ik ben niet heel scherp vandaag 😩. Je hebt gelijk inderdaad. Heb je voorkeur voor een oplossing?

mvdhoek1 commented 1 year ago

My god, ik ben niet heel scherp vandaag 😩. Je hebt gelijk inderdaad. Heb je voorkeur voor een oplossing?

Haha geen probleem, niet persé een voorkeur. Vind jouw oplossing ook een mooie. Denk wel dat mijn oplossing eenvoudiger is omdat die alleen kijkt naar de NAME constant. Daarnaast als je ditzelfde wilt toevoegen bij de leges plugin dan moeten we ook nog een Config class toevoegen om de config waarden van die plugin op te halen bedenk ik mij net. Heb er niet intensief naar gekeken hoor. @SimonvanWijhe wat jij?