feat: add functionality to reassign posts when removing a user from a…#43
feat: add functionality to reassign posts when removing a user from a…#43hnccox-yard wants to merge 2 commits intomainfrom
Conversation
hnccox-yard
requested review from
a team,
Rovasch,
SimonvanWijhe,
dtakken,
ictbeheer,
mvdhoek1 and
rivanuff
and removed request for
a team
|
Is het echt nodig om dit zo uitgebreid te maken. Kunnen we niet gewoon alles re-assignen naar user 0? |
mvdhoek1
left a comment
mvdhoek1
left a comment
There was a problem hiding this comment.
Pas ook even alle snake_case aan naar camelCase.
hnccox-yard
force-pushed
the
feat/reassign_posts_when_removing_user_from_blog
branch
from
5df15c9 to
1315429
Compare
hnccox-yard
requested review from
SimonvanWijhe,
ictbeheer and
mvdhoek1
and removed request for
Rovasch,
SimonvanWijhe,
dtakken,
ictbeheer,
mvdhoek1 and
rivanuff
Nogmaals deze vraag. Tenzij het echt een expliciete wens van de klant is zou ik voor de meest eenvoudige oplossing willen gaan. |
Is een expliciete wens van Jolien, aangezien deze functionaliteit standaard wel aanwezig is bij een blog site wanneer het geen multi-site betreft, en de klant verwacht dezelfde mogelijkheden wanneer wij besluiten om alles als multi-site op te leveren. |
hnccox-yard
force-pushed
the
feat/reassign_posts_when_removing_user_from_blog
branch
4 times, most recently
from
8d7f703 to
1d6faab
Compare
|
hnccox-yard
force-pushed
the
feat/reassign_posts_when_removing_user_from_blog
branch
6 times, most recently
from
8f0632c to
b500876
Compare
hnccox-yard
force-pushed
the
feat/reassign_posts_when_removing_user_from_blog
branch
from
b500876 to
b32ded2
Compare
Translations are working! |
hnccox-yard
requested review from
a team,
Rovasch,
dtakken and
rivanuff
and removed request for
a team
src/User.php
Outdated
| return; | ||
| } | ||
|
|
||
| $userId = (int)$_GET['user']; |
There was a problem hiding this comment.
Ik zou een losse method maken voor de user check met een boolean return type.
There was a problem hiding this comment.
Wat moet ik hier met een boolean?
… blog auto: apply php-cs-fixer changes
hnccox-yard
force-pushed
the
feat/reassign_posts_when_removing_user_from_blog
branch
from
b32ded2 to
e44721d
Compare
5 participants
… blog