BASIRA #254 - Defaulting documents "bookmarks/registers" to "0"

client/src/i18n/en.json

@@ -203,7 +203,7 @@
       "bindingRelationshipToTextBlock": "Binding Relationship to Text Block",
       "bindingType": "Binding Type",
       "bookmarksRegisterColor": "Bookmark/Register Color",
-      "bookmarksRegisters": "Bookmarks/Registers",
+      "bookmarksRegisters": "Number of Bookmarks/Registers Visible",
       "bookmarkRegisterStyle": "Bookmark/Register Style",
       "color": "Color",
       "colorOfForeEdges": "Color of Textblock Edges",
@@ -483,7 +483,7 @@
       "bindingType": "Binding Type",
       "bookmarkRegisterColor": "Color",
       "bookmarkStyle": "Bookmark/Register Style",
-      "bookmarksRegisters": "Bookmarks/Registers",
+      "bookmarksRegisters": "Number of Bookmarks/Registers Visible",
       "decoratedForeEdges": "Decorated Textblock Edges",
       "documentFormat": "Document Format",
       "documentType": "Document Type",

client/src/pages/admin/Document.js

@@ -352,10 +352,12 @@ const Document = (props: Props) => {
                 required={props.isRequired('bookmarks_registers')}
               >
                 <Dropdown
+                  defaultValue={0}
+                  fluid
                   onChange={props.onTextInputChange.bind(this, 'bookmarks_registers')}
                   options={zeroToTenRangeOptionsList}
                   selection
-                  value={props.item.bookmarks_registers || ''}
+                  value={props.item.bookmarks_registers}
                 />
               </Form.Input>
               <ValueListDropdown

db/data/20241127124509_reset_documents_bookmarks_registers.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+class ResetDocumentsBookmarksRegisters < ActiveRecord::Migration[7.0]
+  def up
+    execute <<-SQL.squish
+      UPDATE documents
+         SET bookmarks_registers = 0
+       WHERE bookmarks_registers IS NULL;
+    SQL
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

db/data_schema.rb

@@ -1 +1 @@
-DataMigrate::Data.define(version: 20241127121425)
+DataMigrate::Data.define(version: 20241127124509)

db/migrate/20241127124609_change_documents_bookmarks_registers_default_value.rb

@@ -0,0 +1,5 @@
+class ChangeDocumentsBookmarksRegistersDefaultValue < ActiveRecord::Migration[7.0]
+  def change
+    change_column :documents, :bookmarks_registers, :integer, null: false, default: 0
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.0].define(version: 2024_11_27_121542) do
+ActiveRecord::Schema[7.0].define(version: 2024_11_27_124609) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "pg_stat_statements"
   enable_extension "plpgsql"
@@ -119,7 +119,7 @@
     t.boolean "endband_present", default: false, null: false
     t.boolean "uncut_fore_edges", default: false, null: false
     t.text "fore_edge_text"
-    t.integer "bookmarks_registers", default: 0
+    t.integer "bookmarks_registers", default: 0, null: false
     t.integer "text_columns", default: 0, null: false
     t.boolean "ruling", default: false, null: false
     t.boolean "rubrication", default: false, null: false