dbviewer 0.3.1 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d10de35b5f14855ae9b48f687d54ebc97d5fef12049575ca0c3d2ff05803407
-  data.tar.gz: bf2ced6c5eb2ac2e6c643d3bee0fb0e37e552c9e96f3fbc6a401e760130337bc
+  metadata.gz: 8e6fcb0f5760c3daa24b0bbcd6f7f39d308890386eb7bae0e99b59c5027c0853
+  data.tar.gz: 486266a6f7d882f28b2411860466cb271b7e5d32ca65a25f04bf27601a6fb4c7
 SHA512:
-  metadata.gz: 15fe890300e21acdddcd3d5452bd4ac2ff6d1bb65e889cff892e9c1dca0d16929f2af33bf2b848278353c82c9d0fecfb8a540f40ffc8c58801f953566d3416d0
-  data.tar.gz: 12c1bee1aee7989157c7faf79c69fa101a81412503c762917d91e65d0624444eb4bc86a45cce37ee9a7efb13573542e65a92529be95f333c6522babb192cfb41
+  metadata.gz: 362be7d7fed295db73faad72e0b339d7b1f824d945e7c141c2cb43305d9d31c109526c05fe612135c6cfafc74e618621125a03bab3127ac52ec57a58bfcabbf0
+  data.tar.gz: 0a7289d157e0911d0e5e932c7ebbd964536ade2b125d4de7308fb518516793a6162f690a6d7c0dbc3cb88087754a3b2593265c5c6fa48c9980bb6ca0375cf91c

data/app/controllers/concerns/dbviewer/database_operations.rb

@@ -41,8 +41,8 @@ module Dbviewer
     def fetch_tables_with_stats(include_record_counts = false)
       database_manager.tables.map do |table_name|
         table_stats = {
-          name: table_name,
-          columns_count: database_manager.column_count(table_name)
+          name: table_name
+          # columns_count: database_manager.column_count(table_name)
         }
 
         # Only fetch record counts if explicitly requested
@@ -61,9 +61,7 @@ module Dbviewer
       analytics = {
         total_tables: tables.size,
         total_records: tables.sum { |t| t[:record_count] },
-        total_columns: tables.sum { |t| t[:columns_count] },
-        largest_tables: tables.sort_by { |t| -t[:record_count] }.first(5),
-        widest_tables: tables.sort_by { |t| -t[:columns_count] }.first(5),
+        largest_tables: tables.sort_by { |t| -t[:record_count] }.first(10),
         empty_tables: tables.select { |t| t[:record_count] == 0 }
       }
 

data/app/controllers/dbviewer/entity_relationship_diagrams_controller.rb

@@ -1,8 +1,6 @@
 module Dbviewer
   class EntityRelationshipDiagramsController < ApplicationController
     def index
-      @tables = fetch_tables_with_stats
-
       if @tables.present?
         @table_relationships = fetch_table_relationships
       else
@@ -11,7 +9,7 @@ module Dbviewer
       end
 
       respond_to do |format|
-        format.html # Default to HTML view
+        format.html
         format.json do
           render json: {
             tables: @tables,

data/app/controllers/dbviewer/home_controller.rb

@@ -1,10 +1,13 @@
 module Dbviewer
   class HomeController < ApplicationController
-    skip_before_action :set_tables
-
     def index
-      @tables = fetch_tables_with_stats(include_record_counts: true)
       @analytics = fetch_database_analytics
     end
+
+    private
+
+    def set_tables
+      @tables = fetch_tables_with_stats(include_record_counts: true)
+    end
   end
 end

data/app/controllers/dbviewer/logs_controller.rb

@@ -3,7 +3,7 @@ module Dbviewer
     before_action :set_filters, only: [ :index ]
 
     def index
-      @queries = Dbviewer::Logger.instance.recent_queries(
+      @queries = dbviewer_logger.recent_queries(
         limit: @limit,
         table_filter: @table_filter,
         request_id: @request_id,
@@ -11,16 +11,16 @@ module Dbviewer
       )
 
       if @request_id.present? || @table_filter.present? || @min_duration.present?
-        @stats = Dbviewer::Logger.instance.stats_for_queries(@queries)
+        @stats = dbviewer_logger.stats_for_queries(@queries)
         @filtered_stats = true
       else
-        @stats = Dbviewer::Logger.instance.stats
+        @stats = dbviewer_logger.stats
         @filtered_stats = false
       end
     end
 
     def destroy_all
-      Dbviewer::Logger.instance.clear
+      dbviewer_logger.clear
       flash[:success] = "Query logs cleared successfully"
 
       redirect_to logs_path
@@ -35,5 +35,9 @@ module Dbviewer
       @limit = (params[:limit] || 100).to_i
       @limit = 1000 if @limit > 1000
     end
+
+    def dbviewer_logger
+      @dbviewer_logger ||= Dbviewer::Logger.instance
+    end
   end
 end

data/app/controllers/dbviewer/tables_controller.rb

@@ -3,7 +3,7 @@ module Dbviewer
     include Dbviewer::PaginationConcern
 
     def index
-      @tables = fetch_tables_with_stats
+      @tables = fetch_tables_with_stats(include_record_counts: true)
     end
 
     def show

data/app/views/dbviewer/home/index.html.erb

@@ -65,7 +65,6 @@
                   <tr>
                     <th>Table Name</th>
                     <th class="text-end">Records</th>
-                    <th class="text-end">Columns</th>
                   </tr>
                 </thead>
                 <tbody>
@@ -77,7 +76,6 @@
                         </a>
                       </td>
                       <td class="text-end"><%= number_with_delimiter(table[:record_count]) %></td>
-                      <td class="text-end"><%= table[:columns_count] %></td>
                     </tr>
                   <% end %>
                 </tbody>
@@ -92,47 +90,7 @@
       </div>
     </div>
 
-    <div class="col-md-6 mb-4">
-      <div class="card h-100 shadow-sm">
-        <div class="card-header">
-          <h5 class="card-title mb-0">Tables with Most Columns</h5>
-        </div>
-        <div class="card-body">
-          <% if @analytics[:widest_tables].any? %>
-            <div class="table-responsive">
-              <table class="table table-sm table-hover">
-                <thead>
-                  <tr>
-                    <th>Table Name</th>
-                    <th class="text-end">Columns</th>
-                    <th class="text-end">Records</th>
-                  </tr>
-                </thead>
-                <tbody>
-                  <% @analytics[:widest_tables].each do |table| %>
-                    <tr>
-                      <td>
-                        <a href="<%= dbviewer.table_path(table[:name]) %>">
-                          <%= table[:name] %>
-                        </a>
-                      </td>
-                      <td class="text-end"><%= table[:columns_count] %></td>
-                      <td class="text-end"><%= number_with_delimiter(table[:record_count]) %></td>
-                    </tr>
-                  <% end %>
-                </tbody>
-              </table>
-            </div>
-          <% else %>
-            <div class="text-center my-4 empty-data-message">
-              <p>No table data available</p>
-            </div>
-          <% end %>
-        </div>
-      </div>
-    </div>
-
-    <div class="col-12 mb-4">
+    <div class="col-6 mb-4">
       <div class="card shadow-sm">
         <div class="card-header d-flex justify-content-between align-items-center">
           <h5 class="card-title mb-0">Recent SQL Queries</h5>
@@ -141,7 +99,7 @@
         <div class="card-body p-0">
           <% begin %>
             <% require_dependency "dbviewer/logger" %>
-            <% queries = Dbviewer::Logger.instance.recent_queries(limit: 5) %>
+            <% queries = Dbviewer::Logger.instance.recent_queries(limit: 10) %>
             
             <% if queries.any? %>
               <div class="table-responsive">

data/app/views/dbviewer/tables/index.html.erb

@@ -36,7 +36,7 @@
         <thead>
           <tr>
             <th>Table Name</th>
-            <th class="text-center">Columns</th>
+            <th class="text-end">Records</th>
             <th>Actions</th>
           </tr>
         </thead>
@@ -46,8 +46,8 @@
               <td class="fw-medium">
                 <%= link_to table[:name], table_path(table[:name]), class: "text-decoration-none" %>
               </td>
-              <td class="text-center">
-                <span class="badge bg-secondary-subtle"><%= table[:columns_count] %></span>
+              <td class="text-end">
+                <span class="badge bg-secondary-subtle"><%= table[:record_count] %></span>
               </td>
               <td>
                 <%= link_to raw('<i class="bi bi-eye"></i>'), table_path(table[:name]), class: "btn btn-sm btn-outline-primary", title: "View" %>

data/lib/dbviewer/cache_manager.rb

@@ -0,0 +1,78 @@
+module Dbviewer
+  # CacheManager handles caching concerns for the DatabaseManager
+  # It provides an abstraction layer for managing caches efficiently
+  class CacheManager
+    # Initialize the cache manager
+    # @param config [Dbviewer::Configuration] Configuration object
+    def initialize(config = nil)
+      @config = config || Dbviewer.configuration
+      @dynamic_models = {}
+      @table_columns_cache = {}
+      @table_metadata_cache = {}
+      @cache_last_reset = Time.now
+    end
+
+    # Get a model from cache or return nil
+    # @param table_name [String] Name of the table
+    # @return [Class, nil] The cached model or nil if not found
+    def get_model(table_name)
+      @dynamic_models[table_name]
+    end
+
+    # Store a model in the cache
+    # @param table_name [String] Name of the table
+    # @param model [Class] ActiveRecord model class
+    def store_model(table_name, model)
+      @dynamic_models[table_name] = model
+    end
+
+    # Get column information from cache
+    # @param table_name [String] Name of the table
+    # @return [Array<Hash>, nil] The cached column information or nil if not found
+    def get_columns(table_name)
+      @table_columns_cache[table_name]
+    end
+
+    # Store column information in cache
+    # @param table_name [String] Name of the table
+    # @param columns [Array<Hash>] Column information
+    def store_columns(table_name, columns)
+      @table_columns_cache[table_name] = columns
+    end
+
+    # Get table metadata from cache
+    # @param table_name [String] Name of the table
+    # @return [Hash, nil] The cached metadata or nil if not found
+    def get_metadata(table_name)
+      @table_metadata_cache[table_name]
+    end
+
+    # Store table metadata in cache
+    # @param table_name [String] Name of the table
+    # @param metadata [Hash] Table metadata
+    def store_metadata(table_name, metadata)
+      @table_metadata_cache[table_name] = metadata
+    end
+
+    # Reset caches if they've been around too long
+    def reset_if_needed
+      cache_expiry = @config.cache_expiry || 300
+
+      if Time.now - @cache_last_reset > cache_expiry
+        @table_columns_cache = {}
+        @table_metadata_cache = {}
+        @cache_last_reset = Time.now
+        Rails.logger.debug("[DBViewer] Cache reset due to expiry after #{cache_expiry} seconds")
+      end
+    end
+
+    # Clear all caches - useful when schema changes are detected
+    def clear_all
+      @dynamic_models = {}
+      @table_columns_cache = {}
+      @table_metadata_cache = {}
+      @cache_last_reset = Time.now
+      Rails.logger.debug("[DBViewer] All caches cleared")
+    end
+  end
+end

data/lib/dbviewer/configuration.rb

@@ -48,32 +48,4 @@ module Dbviewer
       @admin_credentials = nil
     end
   end
-
-  # Class accessor for configuration
-  class << self
-    attr_accessor :configuration
-  end
-
-  # Configure the engine with a block
-  #
-  # @example
-  #   Dbviewer.configure do |config|
-  #     config.per_page_options = [10, 25, 50]
-  #     config.default_per_page = 25
-  #   end
-  def self.configure
-    self.configuration ||= Configuration.new
-    yield(configuration) if block_given?
-  end
-
-  # Reset configuration to defaults
-  def self.reset_configuration
-    self.configuration = Configuration.new
-  end
-
-  # Get the current configuration
-  # Creates a default configuration if none exists
-  def self.configuration
-    @configuration ||= Configuration.new
-  end
 end