Create real model for Space

Author: fbacall

app/controllers/application_controller.rb

@@ -108,11 +108,11 @@ def user_not_authorized(exception)
   end
 
   def set_space
-    @current_space = Space.find(request.subdomain) if request.subdomain
+    Space.current_space = Space.find_by_host(request.host)
   end
 
   def current_space
-    @current_space
+    Space.current_space
   end
 
   helper_method :current_space

app/models/default_space.rb

@@ -0,0 +1,23 @@
+class DefaultSpace
+  class Image
+    def url
+      TeSS::Config.site['logo']
+    end
+  end
+
+  def title
+    TeSS::Config.site['title_short']
+  end
+
+  def logo_alt
+    TeSS::Config.site['logo_alt']
+  end
+
+  def image?
+    true
+  end
+
+  def image
+    Image.new
+  end
+end

app/models/space.rb

@@ -1,12 +1,19 @@
-class Space
-  include ActiveModel::Model
+class Space < ApplicationRecord
+  has_image(placeholder: TeSS::Config.placeholder['content_provider'])
 
-  attr_accessor :id, :name, :logo
+  def self.current_space=(space)
+    Thread.current[:current_space] = space
+  end
+
+  def self.current_space
+    Thread.current[:current_space] || Space.default
+  end
+
+  def self.default
+    DefaultSpace.new
+  end
 
-  def self.find(id)
-    return nil if id.nil?
-    if TeSS::Config.spaces&.key?(id)
-      self.new(TeSS::Config.spaces[id].merge(id: id))
-    end
+  def logo_alt
+    "#{title} logo"
   end
 end

app/views/layouts/_header.html.erb

@@ -20,21 +20,13 @@
           <span class="icon-bar"></span>
           <span class="icon-bar"></span>
         </button>
-        <% if current_space %>
-          <a class="navbar-brand" href="/">
-            <% if current_space.logo %>
-              <%= image_tag(current_space.logo, alt: current_space.name) %>
-            <% else %>
-              <%= current_space.name %>
-            <% end %>
-          </a>
-        <% else %>
-          <% if defined? TeSS::Config.site['logo'] and !TeSS::Config.site['logo'].blank? %>
-            <a class="navbar-brand" href="/">
-              <%= image_tag(TeSS::Config.site['logo'], alt: TeSS::Config.site['logo_alt']) %>
-            </a>
+        <a class="navbar-brand" href="/">
+          <% if current_space.image? %>
+            <%= image_tag(current_space.image.url, alt: current_space.logo_alt) %>
+          <% else %>
+            <%= current_space.title %>
           <% end %>
-        <% end %>
+        </a>
       </div>
 
       <div class="collapse navbar-collapse" id="navbar-collapse">

config/tess.example.yml

@@ -159,6 +159,7 @@ default: &default
     bioschemas_testing: false
     learning_paths: false
     collection_curation: true
+    spaces: false
   restrict_content_provider_selection: false
   user_ingestion_methods: ['bioschemas']
   placeholder:

db/migrate/20250204100035_create_spaces.rb

@@ -0,0 +1,14 @@
+class CreateSpaces < ActiveRecord::Migration[7.2]
+  def change
+    create_table :spaces do |t|
+      t.string :title
+      t.text :description
+      t.string :host
+      t.string :theme
+      t.attachment :image
+      t.references :user, foreign_key: true
+      t.timestamps
+      t.index ['host'], name: 'index_spaces_on_host', unique: true
+    end
+  end
+end

db/schema.rb

@@ -474,6 +474,22 @@
     t.index ["user_id"], name: "index_sources_on_user_id"
   end
 
+  create_table "spaces", force: :cascade do |t|
+    t.string "title"
+    t.text "description"
+    t.string "host"
+    t.string "theme"
+    t.string "image_file_name"
+    t.string "image_content_type"
+    t.bigint "image_file_size"
+    t.datetime "image_updated_at"
+    t.bigint "user_id"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["host"], name: "index_spaces_on_host", unique: true
+    t.index ["user_id"], name: "index_spaces_on_user_id"
+  end
+
   create_table "staff_members", force: :cascade do |t|
     t.string "name"
     t.string "role"
@@ -622,6 +638,7 @@
   add_foreign_key "nodes", "users"
   add_foreign_key "sources", "content_providers"
   add_foreign_key "sources", "users"
+  add_foreign_key "spaces", "users"
   add_foreign_key "staff_members", "nodes"
   add_foreign_key "stars", "users"
   add_foreign_key "subscriptions", "users"

test/controllers/static_controller_test.rb

@@ -331,11 +331,11 @@ class StaticControllerTest < ActionController::TestCase
   test 'sets current space based on subdomain' do
     old_host = @request.host
     get :home
-    assert_nil assigns(:current_space)
+    assert_equal 'TTI', Space.current_space.title
 
     @request.host = 'plants.mytess.training'
     get :home
-    assert 'plants', assigns(:current_space).id
+    assert_equal 'TeSS Plants Community', Space.current_space.title
   ensure
     @request.host = old_host
   end

test/fixtures/spaces.yml

@@ -0,0 +1,9 @@
+plants:
+  title: TeSS Plants Community
+  host: plants.mytess.training
+astro:
+  title: TeSS Space Community
+  host: space.mytess.training
+other:
+  title: Other TeSS Community
+  host: other.mytess.training