Merge pull request #214 from UIUCLibrary/issue-213

Always instantiate empty array for variables passed to implode()
anvc · Jun 9, 2023 · 83792e8 · 83792e8
2 parents f173c41 + 16d4234
commit 83792e8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/system/application/models/book_model.php b/system/application/models/book_model.php
@@ -738,8 +738,8 @@ public function save($array=array()) {
 				$this->db->join($this->books_table, $this->pages_table.'.book_id='.$this->books_table.'.book_id');
 				$this->db->where($this->books_table.'.book_id', $book_id);
 				$query = $this->db->get();
+				$book_version_ids = array();
 				if ($query->num_rows()) {
-					$book_version_ids = array();
 					$result = $query->result();
 					foreach ($result as $row) $book_version_ids[] = $row->version_id;
 				}
@@ -749,8 +749,8 @@ public function save($array=array()) {
 				$this->db->join($this->books_table, $this->pages_table.'.book_id='.$this->books_table.'.book_id');
 				$this->db->where($this->books_table.'.book_id', $book_id);
 				$query = $this->db->get();
+				$book_page_ids = array();
 				if ($query->num_rows()) {
-					$book_page_ids = array();
 					$result = $query->result();
 					foreach ($result as $row) $book_page_ids[] = $row->content_id;
 				}