Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plugin defekt für Einrichtungen #34

Closed
tleilax opened this issue Mar 20, 2020 · 2 comments
Closed

Plugin defekt für Einrichtungen #34

tleilax opened this issue Mar 20, 2020 · 2 comments
Assignees
Milestone

Comments

@tleilax
Copy link
Contributor

tleilax commented Mar 20, 2020

Aktiviert man das Plugin in Einrichtungen und erstellt den ersten Raum, so wird in der Tabelle seminare ein neuer Eintrag mit der Id der Einrichtung erstellt. Dies bringt Stud.IP dann aus dem Tritt, weil es nicht mehr korrekt erkennen kann, ob der gewählte Kontext eine Veranstaltung oder eine Einrichtung ist.

@tleilax
Copy link
Contributor Author

tleilax commented Mar 20, 2020

Hintergrund ist der, dass im Plugin scheinbar nur von Veranstaltungen bzw. Models vom Typ Course ausgegangen wird. Auch im Model Meeting werden nur verknüpfte Veranstaltungen erfasst. Leider hat dieses Model ein on_store => store definiert, weshalb es zu dem oben beschriebenen Phänomen kommt, wenn das Meeting gespeichert wird.

Ein Hotfix für Versionen >= 4.2 ist folgender Code in der Hauptklasse des Plugins, um zu verhindern, dass das Meetingsplugin in einer Einrichtung aktiviert werden kann:

    public function isActivatableForContext(Range $context)              
    {
        return get_class($context) === Course::class;
    }

Wünschenswert wäre hier aber eine Anpassung des Plugins, so dass es auch in Einrichtungen zuverlässig aktiviert werden kann. Dafür muss vermutlich an diversen Stellen der Kontext genauer unterschieden werden. Um das genau abschätzen zu können, fehlt mir aber leider der Überblick.

@tgloeggl tgloeggl added this to the 2.0 milestone Mar 23, 2020
@tgloeggl tgloeggl added this to To Do in Priority list May 15, 2020
@tgloeggl tgloeggl removed this from To Do in Priority list Nov 4, 2020
@tgloeggl
Copy link
Member

tgloeggl commented Nov 4, 2020

Wir bauen den von Jan vorgeschlagenen Fix ein

@tgloeggl tgloeggl self-assigned this Nov 4, 2020
@tgloeggl tgloeggl removed this from the 2.0 milestone Dec 9, 2020
@tgloeggl tgloeggl added this to the >=2.44 milestone Mar 22, 2021
@ferishili ferishili self-assigned this Mar 25, 2021
ferishili added a commit that referenced this issue Mar 26, 2021
make plugin available for Course only, fixes #34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants