Sheffield | 26-ITP-Jan | Mahmoud Shaabo | Sprint 2 | Book Library

[mahmoudshaabo1984]

What bugs did I fix?

• Bug 1 - Website loads but doesn't show any books
  The for loop in the render function was missing a closing parenthesis ) before the {. This caused a syntax error that stopped all JavaScript from running.

• Bug 2 - Error in console when you try to add a book
  The submit function was calling library.push(book) but the array is named myLibrary. Fixed by using the correct variable name. Also added validation for the author field.

• Bug 3 - It uses the title name as the author name
  In the submit function, title.value was used twice. The second argument should be author.value.

• Bug 4 - Delete button is broken
  The delete button variable was created as delButton but then referenced as delBut three times. Also the event name was "clicks" instead of "click". Fixed both issues.

• Bug 5 - Read status saves the wrong answer
  The if condition was inverted. check == false was showing "Yes". Fixed so check == true correctly shows "Yes".

How to test

1. Open index.html in the browser
2. Two books appear on load — both show "Yes" in the Read column
3. Click "Add new book" and fill in the form — the new book appears in the table
4. Click the "Yes"/"No" button on any book — it toggles correctly
5. Click "Delete" on any book — the book is removed from the table
6. Try submitting the form with empty fields — an alert appears

Hi @cifarquhar — I have fixed all 5 bugs listed in the readme. Please let me know if you have any feedback. Thank you!

[mahmoudshaabo1984]

Hi @cifarquhar — PR #450 is ready for review.

I have fixed all 5 bugs listed in the readme.md file. The app now loads correctly, books can be added and deleted, and the read status toggles as expected.

Please let me know if anything needs to be changed. Thank you!