Feature #39329
Add a relation between the BE user model and the BE user group model
| Status: | Rejected | Start date: | 2012-07-27 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assignee: | Markus Günther | % Done: | 100% |
|
| Category: | Extbase: Domain | |||
| Target version: | Extbase 6.1 | |||
| Has patch: | No | Tags: | ||
| Votes: | 0 |
Related issues
| related to Extbase MVC Framework - Feature #39324: Add BackendUser.fileMounts | Rejected | 2012-07-27 | ||
| blocked by Extbase MVC Framework - Feature #39332: Add a BackendUserGroup model and repository | Resolved | 2012-07-27 | ||
| blocked by Extbase MVC Framework - Feature #39145: Add a BackEndUser model | Resolved | 2012-07-20 |
Associated revisions
[FEATURE] Add userGroups to backendUser model
This patch adds the new BackendUserGroup model as property to the new
BackendUser model. The patch adds the usual methods for a
SplObjectStorage and the corresponding unit tests
Change-Id: I047361810eeb48c53483c99ed0a79966d0dd0e00
Resolves: #39329
Releases: 6.1
Reviewed-on: https://review.typo3.org/13879
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Tymoteusz Motylewski
Tested-by: Tymoteusz Motylewski
Revert "[FEATURE] Add userGroups to backendUser model"
This reverts commit ba746497220ad5f7244415742c171bb6f8143d3c
Backend user groups are a graph in CMS core. This is not reflected
by the current implementation. The feature is reverted to not add
a half baked solution.
Change-Id: I3b7ee94d5bfe72cc7530144bb681e8a913786a16
Releases: 6.2, 6.1
Related: #39329
Reviewed-on: https://review.typo3.org/20298
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Revert "[FEATURE] Add userGroups to backendUser model"
This reverts commit ba746497220ad5f7244415742c171bb6f8143d3c
Backend user groups are a graph in CMS core. This is not reflected
by the current implementation. The feature is reverted to not add
a half baked solution.
Change-Id: I3b7ee94d5bfe72cc7530144bb681e8a913786a16
Releases: 6.2, 6.1
Related: #39329
Reviewed-on: https://review.typo3.org/20299
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
History
Updated by Markus Günther 10 months ago
- Status changed from New to Accepted
- Target version set to Extbase 6.0
Updated by Markus Günther 10 months ago
- Assignee set to Markus Günther
Updated by Gerrit Code Review 9 months ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13879
Updated by Gerrit Code Review 9 months ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13879
Updated by Ingo Pfennigstorf 8 months ago
- Target version changed from Extbase 6.0 to Extbase 6.1
Updated by Gerrit Code Review 5 months ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13879
Updated by Gerrit Code Review 5 months ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13879
Updated by Gerrit Code Review 5 months ago
Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13879
Updated by Gerrit Code Review 5 months ago
Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13879
Updated by Gerrit Code Review 5 months ago
Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13879
Updated by Gerrit Code Review 5 months ago
Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13879
Updated by Markus Günther 5 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ba746497220ad5f7244415742c171bb6f8143d3c.
Updated by Christian Kuhn 27 days ago
- Status changed from Resolved to Rejected
This was reverted again! Groups are recursive, and the merged solution was only half baked. This gives more problems than it does good :(
Rejected this for now, we probably need a graph solution for this property and others, too.