Log message #4197736

# At Username Text
# Aug 13th 2019, 11:27 neon1024 Seems to be working, as the flash message is `Username or password is incorrect`
# Aug 13th 2019, 11:27 neon1024 I’m just implementing AuthComponent, and I have a `Log::debug` in my `findAuth` finder, but it’s not being called. Anywhere I can debug this?
# Aug 13th 2019, 11:00 admad Yes
# Aug 13th 2019, 10:59 vossen.steven yea no I'm striking out, the visibility of a const is public by default right?
# Aug 13th 2019, 10:54 vossen.steven guess I missed it :O
# Aug 13th 2019, 10:53 dereuromark see rest of docs
# Aug 13th 2019, 10:53 vossen.steven @dereuromark any reason you define const as public?
# Aug 13th 2019, 10:44 d.lisiecki93 i didnt :(
# Aug 13th 2019, 10:44 dereuromark but yeah, it is always fired on patching.
# Aug 13th 2019, 10:43 d.lisiecki93 My problem is that i'm receiving some data from frontend, i'm patching entity and having errors about fields which i'm fullfilling later in `beforeSave` event, but i want to inform frontend about errors
# Aug 13th 2019, 10:43 dereuromark did you try it? :slightly_smiling_face:
# Aug 13th 2019, 10:42 d.lisiecki93 Hi guys, i've got question about table events, is beforeMarshal fired for both new entities / already existing ones?
# Aug 13th 2019, 10:36 vossen.steven I might steal some stuff from your repo there @dereuromark
# Aug 13th 2019, 10:35 vossen.steven I don't use it for roles either, was just a crude example to get my point across :)
# Aug 13th 2019, 10:34 challgren enum + bitmask
# Aug 13th 2019, 10:34 challgren I use the enum tools for statuses, roles are a bit too simple IMHO
# Aug 13th 2019, 10:33 vossen.steven That's actually a lot cleaner, never thought about it tbh, my thought process was, I want to do $entity->status_value, oh easy just get the table and pass the entity status :<
# Aug 13th 2019, 10:30 dereuromark Also nicely connects with Bake and autodetect for IDE etc.
# Aug 13th 2019, 10:30 dereuromark https://github.com/dereuromark/cakephp-tools/blob/master/docs/Entity/Enum.md is IMO the easiest and fastest solution :slightly_smiling_face: if you hardcode it anyways.
# Aug 13th 2019, 10:28 challgren Why not like this?
# Aug 13th 2019, 10:26 slackebot1 string { return TableRegistry::getTableLocator()->get($this->getSource())->types[$this->type]; } ```
# Aug 13th 2019, 10:26 vossen.steven Not sure if I follow but here's a crude example Table ``` const TYPE_USER = 0; const TYPE_ADMIN = 1; /** @var array */ public $types; public function initialize(array $config) { $this->types = [ self::TYPE_USER => __('User'), self::TYPE_ADMIN => __('Administrator'), ]; } ``` Entity ``` /** * @return string */ protected function _getTypeValue():
# Aug 13th 2019, 10:21 challgren Why not an association if you already have the tables
# Aug 13th 2019, 10:19 vossen.steven I only use it for const values, for populating selects/radios and instances where I use a variable table name
# Aug 13th 2019, 10:19 vossen.steven @challgren
# Aug 13th 2019, 10:16 dereuromark A rector task for it would make it a super trivial change. For free.
# Aug 13th 2019, 10:16 neothermic and then I can run the fixer to add the return to all of my $this->redirect calls
# Aug 13th 2019, 10:16 neothermic a lot of my controller testing can't happen until another branch is merged in
# Aug 13th 2019, 10:15 challgren Ahh ok
# Aug 13th 2019, 10:15 neothermic 433 test cases
# Aug 13th 2019, 10:15 challgren Weak sauce!
# Aug 13th 2019, 10:14 neothermic @challgren Currently 764
# Aug 13th 2019, 10:12 challgren Looking in my major project I only use it twice, 1 in an entity because ModelAwareTrait screwed up json encoding it. And the other in a table class afterSave event
# Aug 13th 2019, 10:09 neon1024 Here it is if anyone is interested https://github.com/cakephp/cakephp/pull/10933
# Aug 13th 2019, 10:06 challgren https://github.com/dereuromark/cakephp-queue/blame/master/docs/README.md#L320
# Aug 13th 2019, 10:05 challgren I think dereuromark ’s code taught me to use it honestly
# Aug 13th 2019, 10:04 neon1024 ..and to rename some of the api
# Aug 13th 2019, 10:04 neon1024 It got refactored a while ago to try and remove the static interface
# Aug 13th 2019, 10:04 vossen.steven Yea was just thinking getTableLocator probly uses the same obj
# Aug 13th 2019, 10:04 neon1024 @challgren Well that method will be pulling the instance from the registry if one already exists
# Aug 13th 2019, 10:03 challgren Doesn’t make sense to make a new object when sometimes I use getTableLocator() multiple times