Automatically closed - issue fixed for 2 weeks with no activity.
- 🇯🇵Japan ptmkenny
Drupal 7 is EOL (end of life) so 7.x issues are now being closed.
- 🇬🇧United Kingdom adamps
It seems to be fixed in 4.x. If you still see a problem on 4.x then please reopen the issue.
/** * User ID used for account switcher. * * @var int */ protected $uid;
- 🇳🇱Netherlands megachriz
@bkosborne
Thanks for your contribution. This could be a viable solution too, though it is possible that it will no longer be supported in a future version of PHP (PHP 9?). I'm not sure yet though if it is going to be removed, it's not clear from https://www.php.net/manual/en/language.oop5.properties.php#language.oop5...Meanwhile, I've updated the other MR:
- I've addressed my own remarks from #11.
- I made sure that the data property cannot be get or set directly. An UnexpectedValueException will be thrown when trying to do so. Added test coverage for this change.
- 🇺🇸United States bkosborne New Jersey, USA
I created a new MR that just tells PHP it's OK to have dynamic properties on the item. This is a simple approach. I'm not sure if this is really the best long term solution.
- @bkosborne opened merge request.
- First commit to issue fork.