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

Namespace-, Klassen- und Dateinamen anpassen #86

Merged
merged 3 commits into from
Aug 18, 2024
Merged

Conversation

christophboecker
Copy link
Member

  • Lib-Dateien umbenannt (Dateiname = Klassenname)
  • Cronjobs und deren Dateien umbenannt und in ein Verzeichnis lib/Cronjob gelegt; der alte Name wird beim Update angepasst.
    • FriendsOfRedaxo\Neues\Cronjob\Publish
    • FriendsOfRedaxo\Neues\Cronjob\Sync
  • API-Klassen und deren Dateien umbenannt und in ein Verzeichnis lib/Api gelegt.
    • FriendsOfRedaxo\Neues\Api\Rss
    • FriendsOfRedaxo\Neues\Api\Restful
  • für Api\Rss (ex. neues_rss_api) habe ich extends rex_api_function ergänzt. Soweit mir bekannt sind die API-Klassen immer Erweiterungen von rex_api_function. Warum auch immer das ohne .. extends ... funktioniert hat. Zufall?

@alxndr-w
Copy link
Member

Ich meinte, das ohne "extends" funktionierte, weil ursprünglich nach dem Klassennamen geschaut wurde.

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.

Hier gibt's noch 2 Konflikte, aber erstmal besten Dank für die Arbeit!

@alxndr-w alxndr-w self-requested a review August 18, 2024 00:56
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.

Die Konflikte müssten jetzt korrekt aufgelöst sein.

@alxndr-w alxndr-w merged commit f92131b into main Aug 18, 2024
1 check passed
@alxndr-w alxndr-w deleted the christophboecker-1 branch August 18, 2024 01:02
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