- π¨πSwitzerland znerol
Needs a reroll for 11.x. I'll stick around for reviews here. Find me in the sky lounge at DDD vienna if there are any questions.
- last update
over 1 year ago Patch Failed to Apply
As of
#2347799: Remove bugged session-related methods from AccountInterface β
the AccountInterface
does not have any session related methods anymore. In addition
π
Load user entity in Cookie AuthenticationProvider instead of using manual queries
Needs work
removes the usage UserSession
from production code entirely.
Move the AccountInterface
and AnonymousUser
to \Drupal\Core\Authentication
interface. Replace UserSession
with a User
entity in the remaining test code.
approve change
commit
no
\Drupal\Core\Session\AccountInterface
to \Drupal\Core\Authentication\AccountInterface
, but leave a deprecated subinterface thereof at the old place.\Drupal\Core\Session\AnonymousUserSession
to \Drupal\Core\Authentication\AnonymousUser
.\Drupal\Core\Session\UserSession
and replace it with user entity
π
Load user entity in Cookie AuthenticationProvider instead of using manual queries
Needs work
.\Drupal\Core\Session\AccountProxy
to \Drupal\Core\Authentication\AccountProxy
\Drupal\Core\Session\AccountSwitcher
to \Drupal\Core\Authentication\AccountSwitcher
Follow-up to #2347799: Remove bugged session-related methods from AccountInterface β
Needs work
11.0 π₯
Last updated
A change record needs to be drafted before an issue is committed. Note: Change records used to be called change notifications.
The patch will have to be re-rolled with new suggestions/changes described in the comments in the issue.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Needs a reroll for 11.x. I'll stick around for reviews here. Find me in the sky lounge at DDD vienna if there are any questions.