Log message #4268460

# At Username Text
# Jul 27th 2021, 13:54 cnizzardini or is it just two?
# Jul 27th 2021, 13:54 cnizzardini curious if union types will allow listing three responses?
# Jul 27th 2021, 13:53 dereuromark @brandon I use Response|null|void still everywhere in docblock, which is fine
# Jul 27th 2021, 13:51 cnizzardini ```includes: - phpstan-baseline.neon parameters: level: 6 checkMissingIterableValueType: false checkGenericClassInNonGenericObjectType: false treatPhpDocTypesAsCertain: false bootstrapFiles: - tests/bootstrap.php paths: - src earlyTerminatingMethodCalls: Cake\Console\Shell: - abort```
# Jul 27th 2021, 13:51 cnizzardini I run phpstan with these settings and don't get errors with that
# Jul 27th 2021, 13:50 cnizzardini ``` /** * @return \Cake\Http\Response|null|void Renders view */ public function index()```
# Jul 27th 2021, 13:50 cnizzardini Can you try this:
# Jul 27th 2021, 13:48 slackebot2 $this->getRequest()->getAttribute('paging')[$this->name]['pageCount']]]); } }```
# Jul 27th 2021, 13:48 brandon It is a pretty standard action too. ``` /** * Index method * * @return Response|null */ public function index(): ?Response { $query = $this->Systems->find()->contain(['AssetTags']); try { $this->set('systems', $this->paginate($query)); } catch (NotFoundException $e) { return $this->redirect(['action' => 'index', '?' => ['page' =>
# Jul 27th 2021, 13:44 brandon But like I said, pages are now failing to load so something is wrong (assuming with the upgrade). ```Return value of Inventory\Controller\SystemsController::index() must be an instance of Cake\Http\Response or null, none returned```
# Jul 27th 2021, 13:43 brandon @admad I'm aware and I love it now that I know it exists. I guess I am hellbent on maintaining clean code by making sure phpstan and phpcs come back clean.
# Jul 27th 2021, 13:43 admad (unless it reports a legitimate error)
# Jul 27th 2021, 13:42 admad don't waste time trying to make tools happy
# Jul 27th 2021, 13:42 admad @brandon phpstan is supposed to work for you, not vice versa.
# Jul 27th 2021, 13:40 brandon @dereuromark if I set the phpdoc for a controller action to anything other than Response|null, things like phpstan complain. And now pages will not load unless I return something (I've been returning null. This is new since upgrading to 4.x).
# Jul 27th 2021, 11:12 paolo.bragagni :(
# Jul 27th 2021, 11:12 paolo.bragagni {{ dump(validation) }} there is nothing about associated fields
# Jul 27th 2021, 10:55 paolo.bragagni I have to modify my table.twig to add it?
# Jul 27th 2021, 10:53 paolo.bragagni in my validator
# Jul 27th 2021, 10:53 paolo.bragagni but in my DischiTable theres not a ->notEmpty('author_id')
# Jul 27th 2021, 10:52 paolo.bragagni
# Jul 27th 2021, 10:52 paolo.bragagni AND it shows in bold
# Jul 27th 2021, 10:50 paolo.bragagni null for filed author_id is set to false in DB
# Jul 27th 2021, 10:47 dereuromark make sure to set null=>false here in your schema
# Jul 27th 2021, 10:45 paolo.bragagni where does it get the {%- if fieldData.null %} variable?
# Jul 27th 2021, 10:44 paolo.bragagni It shows as it should not be empty but theres not validator?
# Jul 27th 2021, 10:43 paolo.bragagni it works, but why it not puts forr defaults while I've baked the Table?
# Jul 27th 2021, 10:39 dereuromark so you dont want it to be empty? then set up the validation for it to fail here before it tries to save.#
# Jul 27th 2021, 10:37 paolo.bragagni and not an sql error.. !!
# Jul 27th 2021, 10:37 paolo.bragagni
# Jul 27th 2021, 10:36 paolo.bragagni yes but I'd like to obtain a nice popup like this
# Jul 27th 2021, 10:33 dereuromark optional means: DB field is DEFAULT NULL etc
# Jul 27th 2021, 10:32 dereuromark you cannot omit the author if you told the db that this isn't a possibility. either you decide for being optional or you don't :)
# Jul 27th 2021, 10:30 paolo.bragagni SQLSTATE[HY000]: General error: 1364 Field 'autore_id' doesn't have a default value
# Jul 27th 2021, 10:29 paolo.bragagni if I omit title I got the popup, if I omit the Author (select field) I got a DB error
# Jul 27th 2021, 10:29 paolo.bragagni
# Jul 27th 2021, 10:12 paolo.bragagni solved
# Jul 27th 2021, 09:27 paolo.bragagni {{ action }} ??
# Jul 27th 2021, 09:26 paolo.bragagni how to get the name of the action that is baking in twig?
# Jul 27th 2021, 08:17 dereuromark No, it can also be |void still
# Jul 27th 2021, 06:29 brandon I can't seem to find this documented anywhere - controller actions seem to require `return null;` at the end if you aren't returning a Response to keep the code clean. No examples show this return on actions. What are the suggestions? ```Return value of Inventory\Controller\SystemsController::index() must be an instance of Cake\Http\Response or null, none returned```