Baked Table::buildRules() rules are too brief.
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 35/100
Research direction
Start at the Bake Table::buildRules() template or entry point that produces the shown PHP. Compare its generated rule with the requested named rule and additional properties. Done means generated rules include the requested error metadata and a save() failure reports the error, including in DebugKit.
Written by the indexing model from the issue text.
Description
You get something like this:
$rules->add($rules->isUnique(['email']));
When really we want something like this:
$rules->add(
$rules->isUnique(['email']),
'email-is-unique',
[
'errorField' => 'email',
'message' => 'That email address already exists.',
]
);
Without those additional properties, the rule will cause a save() to fail, but will not report the error anywhere (not even in DebugKit). This is not an acceptable "default" level of quality. We should have the choice to reduce the "verbosity" of the messages, not be forced to add it by rote. That's the whole point of having bake templates, after all.
- Dominant language
- PHP
- Stars
- 2
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from loadsys/CakePHP-LoadsysTheme
-
enhancement
Difficulty 3/5 1-2 days Newbie friendliness 35/100
loadsys/CakePHP-LoadsysTheme#26 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
-
Add Some Recommended Plugins in the Composer.jsonMay be free again @justinyost claimed this 3875 days ago, and no pull request is open. Openenhancement
loadsys/CakePHP-LoadsysTheme#23 · 1 comment · 1 assignee ·
-
enhancement help wanted
Difficulty 5/5 Over a week Newbie friendliness 30/100
-
Update Theme to Match CakePHP 3.2 Updated StylesMay be free again @justinyost claimed this 4038 days ago, and no pull request is open. Openenhancement
loadsys/CakePHP-LoadsysTheme#17 · 1 assignee ·
All issues in loadsys/CakePHP-LoadsysTheme
Similar issues
-
sync-en
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day
-
sync-en
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 4 days
-
Перевод устарел
Difficulty 1/5 Under an hour Newbie friendliness 85/100
-
bug
Difficulty 2/5 Half a day Newbie friendliness 76/100
m3ue/m3u-editor#1604 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
femiwiki/docker-mediawiki#1497 ·
Maintainers usually reply within 1 day