Bug #47115

$roleRepository not initialized correctly

Added by Karsten Dambekalns over 2 years ago. Updated over 2 years ago.

Status:Resolved Start date:2013-04-11
Priority:Must have Due date:
Assigned To:Karsten Dambekalns % Done:

100%

Category:Security
Target version:TYPO3 Flow Base Distribution - 2.0
PHP Version: Complexity:
Has patch:No Affected Flow version:Git 2.0

Description

The $roleRepository is set "manually" in initializeRolesFromPolicy() - only if caches are outdated. But the $roleRepository is needed even if the caches are up to date, so this breaks (one way to see the effect is to try and create a Neos user on the CLI).

Associated revisions

Revision ae01fce3
Added by Karsten Dambekalns over 2 years ago

[BUGFIX] Initialize $roleRepository correctly

The $roleRepository is set "manually" in initializeRolesFromPolicy() -
only if caches are outdated. But the $roleRepository is needed even if
the caches are up to date, so this breaks.

This change moves the initialization outside the check.

Change-Id: Ia7d0261a315a362d52063ee4dbfe04053bde5679
Fixes: #47115
Releases: master, 2.0

Revision f2f334d5
Added by Karsten Dambekalns over 2 years ago

[BUGFIX] Initialize $roleRepository correctly

The $roleRepository is set "manually" in initializeRolesFromPolicy() -
only if caches are outdated. But the $roleRepository is needed even if
the caches are up to date, so this breaks.

This change moves the initialization outside the check.

Change-Id: Ia7d0261a315a362d52063ee4dbfe04053bde5679
Fixes: #47115
Releases: master, 2.0

History

#1 Updated by Gerrit Code Review over 2 years 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 https://review.typo3.org/19792

#2 Updated by Karsten Dambekalns over 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#3 Updated by Gerrit Code Review over 2 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/19827

#4 Updated by Karsten Dambekalns over 2 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF