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

Erste Überarbeitung mit RexStan #90

Merged
merged 4 commits into from
Aug 19, 2024
Merged

Erste Überarbeitung mit RexStan #90

merged 4 commits into from
Aug 19, 2024

Conversation

christophboecker
Copy link
Member

Hier mal die größeren Punkte:

Entry:

  • in getCategories:
    • Rückgabetyp null kommt nicht vor, da $this->getRelatedCollection immer eine Collection liefert; entfernt
    • T (=Category) ergänzt: @return rex_yform_manager_collection<Category>
  • in findOnline:
    • Rückgabetyp null kommt nicht vor, da die Quellen immer eine Collection liefern; entfernt
    • T (=Entry) ergänzt: @return rex_yform_manager_collection<Entry>
  • in findByCategory:
    • Rückgabetyp null kommt nicht vor, da $query->find() immer eine Collection liefert; entfernt
    • T (=Entry) ergänzt: @return rex_yform_manager_collection<Entry>
  • in findByCategoryIds:
    • Rückgabetyp null kommt nicht vor, da $query->find() immer eine Collection liefert; entfernt
    • T (=Entry) ergänzt: @return rex_yform_manager_collection<Entry>
  • in getUrl: Abfrage unnötig; rex_getUrl liefert eh nur Strings

EntryLang:

  • in getEntries:
    • Rückgabetyp null kommt nicht vor, da $this->getRelatedCollection immer eine Collection liefert; entfernt
    • T (=Entry) ergänzt: @return rex_yform_manager_collection<Entry>

Category:

  • in getEntries:
    • Rückgabetyp null kommt nicht vor, da $this->getRelatedCollection mmer eine Collection liefert; entfernt
    • T (=Entry) ergänzt: @return rex_yform_manager_collection<Entry>

RSS:

  • Verweise auf neues_entry umgestellt auf Entry durch @param rex_yform_manager_collection<Entry> $collection
  • Return-Types ergänzt
  • @api ergänzt
  • Überflüssige use entfernt (cs-fixer)

install.php:

  • Wegen "Variable $sql might not be defined." das $sql = rex_sql::factory() nach vorne gezogen
  • rex_addon::get('yform') && rex_addon::get('yform')->isAvailable() reduziert auf rex_addon::get('yform')->isAvailable()
  • rex_addon::get('cronjob') && rex_addon::get('cronjob')->isAvailable() reduziert auf rex_addon::get('cronjob')->isAvailable()
  • rex_addon::get('url') && rex_addon::get('url')->isAvailable() reduziert auf rex_addon::get('uel')->isAvailable()

Dabei fällt dann noch so dies und das auf; teilweise schon mit TODO: ... markiert. More to come ....

@christophboecker christophboecker changed the title Erse Überarbeitung mit RexStan Ersre Überarbeitung mit RexStan Aug 19, 2024
@alxndr-w alxndr-w self-assigned this Aug 19, 2024
@alxndr-w
Copy link
Member

Sauber! Und vielen Dank auch für die Erklärungen dazu, sodass ich das auch in anderen Addons nachziehen kann.

fragments/neues/entry.php Outdated Show resolved Hide resolved
install.php Outdated Show resolved Hide resolved
christophboecker and others added 2 commits August 19, 2024 15:55
Co-authored-by: Alexander Walther <[email protected]>
Co-authored-by: Alexander Walther <[email protected]>
@alxndr-w alxndr-w changed the title Ersre Überarbeitung mit RexStan Erste Überarbeitung mit RexStan Aug 19, 2024
@alxndr-w alxndr-w merged commit 386382d into main Aug 19, 2024
1 check passed
@alxndr-w alxndr-w deleted the cb-2 branch August 19, 2024 14:30
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