Remove AllowDynamicProperties attribute from UserSession

+ protected $langcode; + protected $pass; + protected $status; + protected $created; + protected $changed; + protected $login; + protected $init; + protected $default_langcode; + protected $content_translation_source; + protected $content_translation_outdated; + protected $content_translation_uid; + protected $content_translation_status; + protected $content_translation_created; + protected $rest_test_validation; + public $pass_raw; + public $passRaw; + public $sessionId;

Comments & Activities

Issue created by @andypost
Status changed to Needs review almost 2 years ago3:40pm 27 February 2023
Comment almost 2 years ago →
🇫🇷France andypost
all properties
Status changed to RTBC almost 2 years ago5:35am 28 February 2023
Comment almost 2 years ago →
🇮🇳India Ranjit1032002
I had Reviewed the patch and it's working as expected mentioned in comment #2
Status changed to Needs work almost 2 years ago7:17am 28 February 2023
Comment almost 2 years ago →
🇫🇷France andypost
At least half of this properties are externally set by rest and content translation modules - needs refactoring of usage, removal of dynamic access or replacing userSession with real User entity
Comment almost 2 years ago →
🇨🇭Switzerland berdir Switzerland
That (using user entity) won't be enough. The user password stuff is drupalLogin() related and will likely require is to refactor that to avoid storing arbitrary stuff on either UserSession or content entities, these are just hiding the problem for now.

Remove AllowDynamicProperties attribute from UserSession

Problem/Motivation

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Comments & Activities