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

Tabellenseiten über nur ein Script yform.php darstellen #96

Merged
merged 4 commits into from
Sep 2, 2024
Merged

Conversation

christophboecker
Copy link
Member

Bisher hab es je YForm-Tabelle eine pages/neues.«tabellenseite».php, wobei sich die vier Dateien inhaltlich nicht unterschieden. Das ist nun eingedampft auf eine Datei pages/yform.php, die im jeweiligen Abschnitt der package.yml als subPath: pages\yform.php aufgerufen wird.

Da der zugeordnete Tabellenname nun nicht mehr in der Datei selbst verdrahtet ist, muss er anderweitig festgelegt werden. dazu dient die neue Sektion yform: in der package.yml:

yform:
    neues/entry:
        table_name: FriendsOfRedaxo\Neues\Entry
    neues/category:
        table_name: FriendsOfRedaxo\Neues\Category
    neues/author:
        table_name: FriendsOfRedaxo\Neues\Author
    neues/lang:
        table_name: FriendsOfRedaxo\Neues\EntryLang

Als table_namewird hier die Model-Class angegeben. Möglich wäre auch der Tabellenname ohne Prefix (z.B. neues_entry). Aber hier konsequent wie im ganzen Addon nur die Model-Class.

Da bisher Titel/menü des Addons aus den einzelnen Seiten-Dateien aufgebaut wurde statt wie bei den meisten anderen Addons, wurd auch hier eine Angleichung vorgenommen. Titel/Menü werden nun wieder aus der übergeordneten DAtei neues.php ausgegeben.

Grund für die bisherigen Lösung: für die Hilfeseiten wurde dynamisch ein Submenü generiert, dass aber entsprechenden Code VOR der Ausgabe des Titels benötigt. Damit kein doppelter Titel ausgegeben wird, hat neues.php eine seitenbezogene Weiche bekommen.

package.yml Outdated Show resolved Hide resolved
Copy link
Member

@alxndr-w alxndr-w left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alles prima, bis auf eine Kleinigkeit zum TODO.

Co-authored-by: Alexander Walther <[email protected]>
@alxndr-w alxndr-w merged commit f462f1d into main Sep 2, 2024
1 check passed
@alxndr-w alxndr-w deleted the cb-6 branch September 2, 2024 09:42
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

Successfully merging this pull request may close these issues.

2 participants