Commit fb50124

justin808claude

and

committed

Fix validation logic to show all error messages and add explicit class name

- Change validation logic from if/else chain to independent if statements This ensures both "Author" and "Text" error messages are displayed when both fields are blank, rather than just showing one error at a time - Add explicit class name CommentsController for better debugging and compatibility with future transpiler changes 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>

1 parent 577e4b5 commit fb50124Copy full SHA for fb50124

File tree

1 file changed

-5

lines changed

client/app/controllers
- comments_controller.js

1 file changed

-5

lines changed

`‎client/app/controllers/comments_controller.js‎`

Lines changed: 3 additions & 5 deletions

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ import { mangle } from 'marked-mangle';`
`7`	`7`	`marked.use(gfmHeadingId());`
`8`	`8`	`marked.use(mangle());`
`9`	`9`
`10`		`-export default class extends Controller {`
	`10`	`+export default class CommentsControllerextends Controller {`
`11`	`11`	`static targets = ['commentList', 'commentAuthor', 'commentText', 'commentRefresh', 'alertDiv', 'errorList'];`
`12`	`12`
`13`	`13`	`resetText() {`
`@@ -22,10 +22,8 @@ export default class extends Controller {`
`22`	`22`	`errorList.innerHTML = '';`
`23`	`23`	`if (!inputAuthor.value) {`
`24`	`24`	`errors.push('Author');`
`25`		`- } else if (!inputText.value) {`
`26`		`- errors.push('Text');`
`27`		`- } else {`
`28`		`- errors.push('Author');`
	`25`	`+ }`
	`26`	`+ if (!inputText.value) {`
`29`	`27`	`errors.push('Text');`
`30`	`28`	`}`
`31`	`29`	`errors.forEach((error) => {`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit fb50124

File tree

1 file changed

1 file changed

`‎client/app/controllers/comments_controller.js‎`

0 commit comments