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

Getter/Setter für ycom_user und ycom_group ergänzt #481

Closed
wants to merge 3 commits into from

Conversation

alxndr-w
Copy link
Collaborator

@alxndr-w alxndr-w commented Apr 4, 2024

Alle Felder, die auch bei Installation installiert / sichergestellt werden, erhalten korrespondierende Methoden.

@alxndr-w alxndr-w requested a review from dergel April 4, 2024 19:20
@alxndr-w alxndr-w self-assigned this Apr 4, 2024
@dergel
Copy link
Member

dergel commented Aug 29, 2024

ich schreibs lieber allgemein. Danke erstmal :) ...
Bitte die Bools die als 1 und 0 gespeichert werden auch immer als book zurückgeben bei get und bei set nur bool übergeben und als 1 und 0 speichern. Das wäre toll

@alxndr-w
Copy link
Collaborator Author

Verstehe ich!

Aber: Bspw. termsofuse_accepted ist ein Zeitstempel, sollten dafür nicht im Zweifel neue Methoden wie hasAccepted() verwendet werden, die dann bool zurückliefern?

@dergel
Copy link
Member

dergel commented Aug 29, 2024

termsofuse_accepted ist ein Bool also 1/0 in der Datenbank. Das heisst es würde doch passen

lib/ycom_user.php Outdated Show resolved Hide resolved
@alxndr-w
Copy link
Collaborator Author

@dergel wie im Meeting besprochen, du möchtest gerne PRs selbst umsetzen, solche Anliegen lieber als Issues. Daher würde ich hier zumachen.

@alxndr-w alxndr-w closed this Nov 15, 2024
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