Skip to content

Commit ff68704

Browse files
committed
Merge branch 'book-library' of https://github.com/Bluejay600/Module-Data-Flows into book-library
2 parents e99d6aa + c830178 commit ff68704

File tree

1 file changed

+14
-10
lines changed

1 file changed

+14
-10
lines changed

debugging/book-library/script.js

Lines changed: 14 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -31,14 +31,16 @@ function submit() {
3131
if (
3232
title.value == null ||
3333
title.value == "" ||
34-
pages.value == null ||
35-
pages.value == ""
34+
pages.value == null ||
35+
pages.value == "" ||
36+
author.value == null ||
37+
author.value == ""
3638
) {
3739
alert("Please fill all fields!");
3840
return false;
3941
} else {
40-
let book = new Book(title.value, title.value, pages.value, check.checked);
41-
library.push(book);
42+
let book = new Book(title.value, author.value, pages.value, check.checked);
43+
myLibrary.push(book);
4244
render();
4345
}
4446
}
@@ -53,6 +55,8 @@ function Book(title, author, pages, check) {
5355
function render() {
5456
let table = document.getElementById("display");
5557
let rowsNumber = table.rows.length;
58+
console.log(myLibrary)
59+
5660
//delete old table
5761
for (let n = rowsNumber - 1; n > 0; n--) {
5862
table.deleteRow(n);
@@ -76,7 +80,7 @@ function render() {
7680
changeBut.className = "btn btn-success";
7781
wasReadCell.appendChild(changeBut);
7882
let readStatus = "";
79-
if (myLibrary[i].check == false) {
83+
if (myLibrary[i].check == true) {
8084
readStatus = "Yes";
8185
} else {
8286
readStatus = "No";
@@ -90,11 +94,11 @@ function render() {
9094

9195
//add delete button to every row and render again
9296
let delButton = document.createElement("button");
93-
delBut.id = i + 5;
94-
deleteCell.appendChild(delBut);
95-
delBut.className = "btn btn-warning";
96-
delBut.innerHTML = "Delete";
97-
delBut.addEventListener("clicks", function () {
97+
delButton.id = i + 5;
98+
deleteCell.appendChild(delButton);
99+
delButton.className = "btn btn-warning";
100+
delButton.innerHTML = "Delete";
101+
delButton.addEventListener("click", function () {
98102
alert(`You've deleted title: ${myLibrary[i].title}`);
99103
myLibrary.splice(i, 1);
100104
render();

0 commit comments

Comments
 (0)