remove onloan column

database/migrations/mysql/02-remove-onloan.sql

@@ -0,0 +1 @@
+ALTER TABLE books DROP COLUMN onloan;

database/mysql.go

@@ -155,7 +155,7 @@ func (m *MySQL) GetAllBooks(ctx context.Context) ([]media.Book, error) {
 	allBooksQuery := fmt.Sprintf(`SELECT
 		id, title, authors, sortauthor, isbn10, isbn13, format,
 		genre, publisher, series, volume, year, signed,
-		description, notes, onloan, coverurl
+		description, notes, coverurl
 	FROM %s`, m.tableName)
 
 	books := []media.Book{}
@@ -174,7 +174,7 @@ func (m *MySQL) GetAllBooks(ctx context.Context) ([]media.Book, error) {
 			&b.Format, &b.Genre, &b.Publisher,
 			&b.Series, &b.Volume, &b.Year,
 			&b.Signed, &b.Description, &b.Notes,
-			&b.OnLoan, &b.CoverURL)
+			&b.CoverURL)
 		if err != nil {
 			return nil, err
 		}
@@ -192,9 +192,9 @@ func (m *MySQL) AddBook(ctx context.Context, b *media.Book) error {
 
 	res, err := m.connection.ExecContext(ctx, `
 		INSERT INTO `+m.tableName+`
-		(title, authors, sortauthor, isbn10, isbn13, format, genre, publisher, series, volume, year, signed, description, notes, onloan, coverurl)
+		(title, authors, sortauthor, isbn10, isbn13, format, genre, publisher, series, volume, year, signed, description, notes, coverurl)
 		VALUES
-		(?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`,
+		(?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`,
 		b.Title,
 		strings.Join(b.Authors, ";"),
 		b.SortAuthor,
@@ -209,7 +209,6 @@ func (m *MySQL) AddBook(ctx context.Context, b *media.Book) error {
 		b.Signed,
 		b.Description,
 		b.Notes,
-		b.OnLoan,
 		b.CoverURL,
 	)
 	if err != nil {
@@ -250,7 +249,6 @@ func (m *MySQL) UpdateBook(ctx context.Context, old, new *media.Book) error {
 			signed=?
 			description=?
 			notes=?
-			onloan=?
 			coverurl=?
 		WHERE id=?`,
 		new.Title,
@@ -267,7 +265,6 @@ func (m *MySQL) UpdateBook(ctx context.Context, old, new *media.Book) error {
 		new.Signed,
 		new.Description,
 		new.Notes,
-		new.OnLoan,
 		new.CoverURL,
 		old.ID)
 	if err != nil {

frontend/files/app.js

@@ -127,7 +127,6 @@ function BookTemplate({
   description,
   format,
   notes,
-  onLoan,
   publisher,
   series,
   signed,
@@ -136,7 +135,7 @@ function BookTemplate({
   year,
 }) {
   return `${coverURL ? `<img src="${coverURL}"/>` : ""}
-  <h1 ${onLoan ? "class='onLoan' " : ""}>${title}</h1>
+  <h1>${title}</h1>
   <h2>${authors}</h2>
   <span>${isbn13}</span><br/>
   <span>${publisher}, ${year}</span><br/>
@@ -147,7 +146,6 @@ function BookTemplate({
   }
   ${signed ? "<span>Signed by the author ✒</span><br/>" : ""}
   <span>${format}</span>
-  ${onLoan ? `<h2 class="onLoan">On loan to ${onLoan}</h2>` : ""}
   <div class="description">
     <p>${description}</p>
     ${notes ? `<span>Notes:</span><p>${notes}</p>` : ""}
@@ -157,14 +155,13 @@ function BookTemplate({
 function TableRowTemplate({
   "isbn-13": isbn13,
   authors,
-  onLoan,
   publisher,
   rowNumber,
   signed,
   title,
   year,
 }) {
-  return `<tr class="tRow ${onLoan ? "onLoan" : ""}" id="${rowNumber}">
+  return `<tr class="tRow id="${rowNumber}">
     <td class="title">
       ${title} ${
     signed ? '<span class="signed" title="Signed by the author" >✒</span>︎' : ""

frontend/files/records/style.css

@@ -243,10 +243,6 @@ body {
   cursor: pointer;
 }
 
-.recordTable .onLoan {
-  color: #bbb;
-}
-
 .recordTable .tRow .name {
   font-style: italic;
   max-width: 600px;
@@ -274,11 +270,3 @@ body {
 #current .description p {
   padding: 20px 0;
 }
-
-#current h1.onLoan {
-  color: #bbb;
-}
-
-#current h2.onLoan {
-  font-weight: bold;
-}

frontend/files/style.css

@@ -243,10 +243,6 @@ body {
   cursor: pointer;
 }
 
-.bookTable .onLoan {
-  color: #bbb;
-}
-
 .bookTable .tRow .title {
   font-style: italic;
   max-width: 600px;
@@ -274,11 +270,3 @@ body {
 #current .description p {
   padding: 20px 0;
 }
-
-#current h1.onLoan {
-  color: #bbb;
-}
-
-#current h2.onLoan {
-  font-weight: bold;
-}

importer/csv.go

@@ -40,7 +40,6 @@ func CSVToBooks(r io.Reader) ([]media.Book, error) {
 			Signed:      row[hmap["signed"]] == "yes", // convert from known string to bool
 			Description: row[hmap["description"]],
 			Notes:       row[hmap["notes"]],
-			OnLoan:      row[hmap["onloan"]],
 			CoverURL:    row[hmap["coverurl"]],
 		}
 		books = append(books, b)

media/media.go

@@ -16,7 +16,6 @@ type Book struct {
 	Signed      bool     `json:"signed"`
 	Description string   `json:"description"`
 	Notes       string   `json:"notes"`
-	OnLoan      string   `json:"onLoan"`
 	CoverURL    string   `json:"coverURL"`
 }
 

ui/ui.go

@@ -404,7 +404,6 @@ func (l *BookDetails) Draw(s tcell.Screen) {
 		{"Volume", l.book.Volume},
 		{"Year", l.book.Year},
 		{"Signed", strconv.FormatBool(l.book.Signed)},
-		{"On Loan", l.book.OnLoan},
 		{"Cover URL", l.book.CoverURL},
 		{"Notes", l.book.Notes},
 		{"Description", l.book.Description},