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

array_converter mit Attributen konfigurieren #136

Merged
merged 5 commits into from
Nov 19, 2024
Merged

Conversation

MHajoha
Copy link
Member

@MHajoha MHajoha commented Oct 22, 2024

  • $config->rename wird ersetzt durch #[array_key(...)]
  • $config->alias wird ersetzt durch #[array_alias(...)]
  • $config->array_elements wird ersetzt durch #[array_element_class(...)]
  • $config->discriminated_by, $config->variant und $config->fallback_variant wird ersetzt durch #[array_polymorphic(...)]

Es wird natürlich einige Konflikte mit 4e273bc geben, die würde ich dann fixen, sobald das gemergt ist.

Copy link
Contributor

@janbritz janbritz left a comment

Choose a reason for hiding this comment

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

Ziemlich nice :D

classes/form/elements/checkbox_element.php Outdated Show resolved Hide resolved
classes/package/package.php Outdated Show resolved Hide resolved
@MHajoha MHajoha force-pushed the array-converter-attrs branch from d7b8c33 to e30413d Compare October 23, 2024 16:17
Copy link

@larsbonczek larsbonczek left a comment

Choose a reason for hiding this comment

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

Du verwendest überall double quotes für Strings. Der Coding Style von Moodle erlaubt das eigentlich nur in Sonderfällen: https://moodledev.io/general/development/policies/codingstyle#strings

@MHajoha MHajoha force-pushed the array-converter-attrs branch from cb9dfaa to 069d7b8 Compare October 28, 2024 14:36
@MHajoha
Copy link
Member Author

MHajoha commented Oct 28, 2024

Du verwendest überall double quotes für Strings. Der Coding Style von Moodle erlaubt das eigentlich nur in Sonderfällen: https://moodledev.io/general/development/policies/codingstyle#strings

Cringe. Es passt eigentlich nicht in den PR, aber ich habe es in zwei neuen Commits für das ganze Repo angepasst (danke phpcbf). Kann ich gerne stattdessen auf dev machen, wenn euch das den PR zu sehr zumüllt.

@MHajoha MHajoha force-pushed the array-converter-attrs branch from 069d7b8 to c87a889 Compare November 19, 2024 13:58
@MHajoha MHajoha merged commit fd316b3 into dev Nov 19, 2024
6 checks passed
@MHajoha MHajoha deleted the array-converter-attrs branch November 19, 2024 16:37
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.

4 participants