prailroady 1.5.3 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ed0b7fa52bd77b6ece893c54707a79f84bf51e3216207a98b45d5cbea73bdea
-  data.tar.gz: e1b14f97c09ab9cafff1185a1da5e77718dd9f12f71640d32a50e4f269d07dc5
+  metadata.gz: db58ff716ee21346c67b6382fbcd48eb30667aca10de1f44ca684b722515270a
+  data.tar.gz: 21becfea99d5810f2824beed8a44e0ac433fc25c70302aa2ff88d280e184257d
 SHA512:
-  metadata.gz: 450c28b442b5b06de78f189e17bf6825bf7bb11d44c3d29e77e15a260ada0182615125dee505e8e0f618e7f2615c5f9ec191ff2b822128d567c9e46ec7b8c709
-  data.tar.gz: 53515410d1d61e76a9cf6c6956da1ee92676b608ba436f92379250793a271df47e0dbb24fc2e7f27bd5824dc2e728bebca75e67876e40225cd45868fdf153e2e
+  metadata.gz: 6fee736b23e129288f01fd765b1fd7518f34802d0b58373a8927b7a95ea63dc49d90a61d444e89339c266d2f9284fc8ed4a6a88f4d041cc442e894f07f1cf9a8
+  data.tar.gz: 8816a14be17885d1820f4e515a45998301e3c90e167b8df2927b47ac32d57952d214af6c064d8e613570fd08253c115e2f7aef9bec3a1e90154ca208cf2c3fca

data/.travis.yml

@@ -1,3 +1,3 @@
 language: ruby
 rvm:
- - 2.4.1
+ - 3.0.2

data/AUTHORS.rdoc

@@ -18,6 +18,6 @@ http://railroady.prestonlee.com
 * Tim Harvey
 * Atli Christiansen
 * John Bintz (http://www.coswellproductions.com/)
-
+* Stefan Wrobel
 
 And of course, many thanks to the many patch submitters and testers that make this possible!

data/Gemfile

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 gemspec

data/Gemfile.lock

@@ -6,21 +6,21 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (5.1.1)
+    activesupport (7.0.2.3)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (~> 0.7)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-    concurrent-ruby (1.0.5)
-    i18n (0.8.4)
-    minitest (5.10.2)
-    rake (12.0.0)
-    thread_safe (0.3.6)
-    tzinfo (1.2.3)
-      thread_safe (~> 0.1)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+    concurrent-ruby (1.1.10)
+    i18n (1.10.0)
+      concurrent-ruby (~> 1.0)
+    minitest (5.15.0)
+    rake (13.0.6)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
 
 PLATFORMS
-  ruby
+  arm64-darwin-21
 
 DEPENDENCIES
   activesupport
@@ -30,4 +30,4 @@ DEPENDENCIES
   rake
 
 BUNDLED WITH
-   1.16.6
+   2.2.31

data/README.md

@@ -1,3 +1,7 @@
+# add gem to Gemfile
+
+`gem 'prailroady'`
+
 # Use online preview
 
 https://planttext.com/

data/bin/prailroady

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 # PrailRoady - RoR diagrams generator
 # http://github.com/preston/railroady
@@ -25,7 +26,8 @@ APP_HUMAN_NAME = 'PrailRoady'
 APP_VERSION    = PrailRoady::VERSION
 COPYRIGHT      = 'Copyright (C) 2007-2008 Javier Smaldone, 2009 Peter Hoeg, 2010-2015 Preston Lee'
 
-options = OptionsStruct.new(app_name: APP_NAME, app_human_name: APP_HUMAN_NAME, app_version: APP_VERSION, copyright: COPYRIGHT)
+options = OptionsStruct.new(app_name: APP_NAME, app_human_name: APP_HUMAN_NAME, app_version: APP_VERSION,
+                            copyright: COPYRIGHT)
 
 options.parse ARGV
 
@@ -44,7 +46,7 @@ when 'controllers'
 when 'aasm'
   diagram = AasmDiagram.new(options)
 else
-  STDERR.print "#{APP_HUMAN_NAME} v#{APP_VERSION}\n" \
+  $stderr.print "#{APP_HUMAN_NAME} v#{APP_VERSION}\n" \
     "Error: You must supply a command\n" \
     "  (try #{APP_NAME} -h)\n\n"
   exit 1

data/lib/prailroady/aasm_diagram.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # PrailRoady - RoR diagrams generator
 # http://railroad.rubyforge.org
 #
@@ -20,18 +21,17 @@ class AasmDiagram < AppDiagram
 
   # Process model files
   def generate
-    STDERR.print "Generating AASM diagram\n" if @options.verbose
+    $stderr.print "Generating AASM diagram\n" if @options.verbose
     get_files.each do |f|
       process_class extract_class_name(f).constantize
     end
   end
 
   def get_files(prefix = '')
-    files = !@options.specify.empty? ? Dir.glob(@options.specify) : Dir.glob(prefix + 'app/models/**/*.rb')
+    files = !@options.specify.empty? ? Dir.glob(@options.specify) : Dir.glob("#{prefix}app/models/**/*.rb")
     files += Dir.glob('vendor/plugins/**/app/models/*.rb') if @options.plugins_models
-    files -= Dir.glob(prefix + 'app/models/concerns/**/*.rb') unless @options.include_concerns
-    files -= Dir.glob(@options.exclude)
-    files
+    files - Dir.glob("#{prefix}app/models/concerns/**/*.rb") unless @options.include_concerns
+    Dir.glob(@options.exclude)
   end
 
   private
@@ -45,49 +45,25 @@ class AasmDiagram < AppDiagram
     enable_stdout
     print_error 'model classes'
     raise
-  end  # load_classes
+  end
 
   # Process a model class
   def process_class(current_class)
-    STDERR.print "\tProcessing #{current_class}\n" if @options.verbose
+    $stderr.print "\tProcessing #{current_class}\n" if @options.verbose
 
-    # Only interested in acts_as_state_machine models.
-    process_acts_as_state_machine_class(current_class)  if current_class.respond_to?(:states)
+    # Only interested in aasm models.
     process_aasm_class(current_class)  if current_class.respond_to?(:aasm_states) || current_class.respond_to?(:aasm)
-  end # process_class
-
-  def process_acts_as_state_machine_class(current_class)
-    node_attribs = []
-    node_type = 'aasm'
-
-    STDERR.print "\t\tprocessing as acts_as_state_machine\n" if @options.verbose
-    current_class.aasm.states.each do |state_name|
-      node_shape = (current_class.aasm.initial_state == state_name) ? ', peripheries = 2' : ''
-      node_attribs << "#{current_class.name.downcase}_#{state_name} [label=#{state_name} #{node_shape}];"
-    end
-    @graph.add_node [node_type, current_class.name, node_attribs]
-
-    current_class.aasm.events.each do |event|
-      event_name = event.name
-      event.transitions.each do |transition|
-        @graph.add_edge [
-          'event',
-          current_class.name.downcase + '_' + transition.from.to_s,
-          current_class.name.downcase + '_' + transition.to.to_s,
-          event_name.to_s
-        ]
-      end
-    end
   end
 
   def process_aasm_class(current_class)
     node_attribs = []
     node_type = 'aasm'
+    diagram_friendly_class_name = current_class.name.downcase.gsub(/[^a-z0-9\-_]+/i, '_')
 
-    STDERR.print "\t\tprocessing as aasm\n" if @options.verbose
+    $stderr.print "\t\tprocessing as aasm\n" if @options.verbose
     current_class.aasm.states.each do |state|
-      node_shape = (current_class.aasm.initial_state == state.name) ? ', peripheries = 2' : ''
-      node_attribs << "#{current_class.name.downcase}_#{state.name} [label=#{state.name} #{node_shape}];"
+      node_shape = current_class.aasm.initial_state == state.name ? ', peripheries = 2' : ''
+      node_attribs << "#{diagram_friendly_class_name}_#{state.name} [label=#{state.name} #{node_shape}];"
     end
     @graph.add_node [node_type, current_class.name, node_attribs]
 
@@ -95,11 +71,11 @@ class AasmDiagram < AppDiagram
       event.transitions.each do |transition|
         @graph.add_edge [
           'event',
-          current_class.name.downcase + '_' + transition.from.to_s,
-          current_class.name.downcase + '_' + transition.to.to_s,
+          "#{diagram_friendly_class_name}_#{transition.from}",
+          "#{diagram_friendly_class_name}_#{transition.to}",
           event.name.to_s
         ]
       end
     end
   end
-end # class AasmDiagram
+end

data/lib/prailroady/app_diagram.rb

@@ -18,25 +18,25 @@ class AppDiagram
   # Print diagram
   def print
     if @options.output
-      old_stdout = STDOUT.dup
+      old_stdout = $stdout.dup
       begin
-        STDOUT.reopen(@options.output)
+        $stdout.reopen(@options.output)
       rescue
-        STDERR.print "Error: Cannot write diagram to #{@options.output}\n\n"
+        $stderr.print "Error: Cannot write diagram to #{@options.output}\n\n"
         exit 2
       end
     end
 
     if @options.xmi
-      STDERR.print "Generating XMI diagram\n" if @options.verbose
-      STDOUT.print @graph.to_xmi
+      $stderr.print "Generating XMI diagram\n" if @options.verbose
+      $stdout.print @graph.to_xmi
     else
-      STDERR.print "Generating DOT graph\n" if @options.verbose
-      STDOUT.print @graph.to_dot
+      $stderr.print "Generating DOT graph\n" if @options.verbose
+      $stdout.print @graph.to_dot
     end
 
-    STDOUT.reopen(old_stdout) if @options.output
-  end # print
+    $stdout.reopen(old_stdout) if @options.output
+  end
 
   def process
     load_environment
@@ -61,7 +61,7 @@ class AppDiagram
 
   # Load Rails application's environment
   def load_environment
-    STDERR.print "Loading application environment\n" if @options.verbose
+    $stderr.print "Loading application environment\n" if @options.verbose
     begin
       disable_stdout
       l = File.join(Dir.pwd.to_s, @options.config_file)
@@ -72,24 +72,24 @@ class AppDiagram
       print_error 'application environment'
       raise
     end
-    STDERR.print "Loading application classes as we go\n" if @options.verbose
+    $stderr.print "Loading application classes as we go\n" if @options.verbose
   end
 
   # Prevents Rails application from writing to STDOUT
   def disable_stdout
-    @old_stdout = STDOUT.dup
+    @old_stdout = $stdout.dup
     # via  Tomas Matousek, http://www.ruby-forum.com/topic/205887
-    STDOUT.reopen(::RUBY_PLATFORM =~ /djgpp|(cyg|ms|bcc)win|mingw/ ? 'NUL' : '/dev/null')
+    $stdout.reopen(::RUBY_PLATFORM =~ /djgpp|(cyg|ms|bcc)win|mingw/ ? 'NUL' : '/dev/null')
   end
 
   # Restore STDOUT
   def enable_stdout
-    STDOUT.reopen(@old_stdout)
+    $stdout.reopen(@old_stdout)
   end
 
   # Print error when loading Rails application
   def print_error(type)
-    STDERR.print "Error loading #{type}.\n  (Are you running " \
+    $stderr.print "Error loading #{type}.\n  (Are you running " \
                  "#{@options.app_name} on the application's root directory?)\n\n"
   end
 
@@ -100,4 +100,4 @@ class AppDiagram
     # File.basename(filename).chomp(".rb").camelize
     filename.split('/')[2..-1].collect(&:camelize).join('::').chomp('.rb')
   end
-end # class AppDiagram
+end

data/lib/prailroady/controllers_diagram.rb

@@ -20,7 +20,7 @@ class ControllersDiagram < AppDiagram
 
   # Process controller files
   def generate
-    STDERR.print "Generating controllers diagram\n" if @options.verbose
+    $stderr.print "Generating controllers diagram\n" if @options.verbose
     files = get_files
     # only add APP_CONTROLLER if it isn't already included from the glob above
     files << APP_CONTROLLER unless files.include? APP_CONTROLLER
@@ -31,10 +31,10 @@ class ControllersDiagram < AppDiagram
       begin
         process_class class_name.constantize
       rescue Exception
-        STDERR.print "Warning: exception #{$ERROR_INFO} raised while trying to load controller class #{f}"
+        $stderr.print "Warning: exception #{$ERROR_INFO} raised while trying to load controller class #{f}"
       end
     end
-  end # generate
+  end
 
   def get_files(prefix = '')
     files = !@options.specify.empty? ? Dir.glob(@options.specify) : Dir.glob(prefix << 'app/controllers/**/*_controller.rb')
@@ -48,7 +48,7 @@ class ControllersDiagram < AppDiagram
   end
 
   def extract_class_name(filename)
-    filename.match(/.*\/controllers\/(.*).rb$/)[1].camelize
+    filename.match(%r{.*/controllers/(.*).rb$})[1].camelize
   end
 
   private
@@ -64,11 +64,11 @@ class ControllersDiagram < AppDiagram
     enable_stdout
     print_error 'controller classes'
     raise
-  end # load_classes
+  end
 
   # Proccess a controller class
   def process_class(current_class)
-    STDERR.print "\tProcessing #{current_class}\n" if @options.verbose
+    $stderr.print "\tProcessing #{current_class}\n" if @options.verbose
 
     if @options.brief
       @graph.add_node ['controller-brief', current_class.name]
@@ -77,15 +77,21 @@ class ControllersDiagram < AppDiagram
       node_attribs = { public: [],
                        protected: [],
                        private: [] }
-      current_class.public_instance_methods(false).sort.each do |m|
-        node_attribs[:public] << m
-      end unless @options.hide_public
-      current_class.protected_instance_methods(false).sort.each do |m|
-        node_attribs[:protected] << m
-      end unless @options.hide_protected
-      current_class.private_instance_methods(false).sort.each do |m|
-        node_attribs[:private] << m
-      end unless @options.hide_private
+      unless @options.hide_public
+        current_class.public_instance_methods(false).sort.each do |m|
+          node_attribs[:public] << m
+        end
+      end
+      unless @options.hide_protected
+        current_class.protected_instance_methods(false).sort.each do |m|
+          node_attribs[:protected] << m
+        end
+      end
+      unless @options.hide_private
+        current_class.private_instance_methods(false).sort.each do |m|
+          node_attribs[:private] << m
+        end
+      end
       @graph.add_node ['controller', current_class.name, node_attribs]
     elsif @options.modules && current_class.is_a?(Module)
       @graph.add_node ['module', current_class.name]
@@ -95,9 +101,9 @@ class ControllersDiagram < AppDiagram
     if @options.inheritance && (transitive_subclasses_of(ApplicationController).include? current_class)
       @graph.add_edge ['is-a', current_class.superclass.name, current_class.name]
     end
-  end # process_class
+  end
 
   def transitive_subclasses_of(klass)
     klass.subclasses | klass.subclasses.map { |subklass| transitive_subclasses_of(subklass) }.flatten
   end
-end # class ControllersDiagram
+end

data/lib/prailroady/models_diagram.rb

@@ -17,20 +17,20 @@ class ModelsDiagram < AppDiagram
 
   # Process model files
   def generate
-    STDERR.puts 'Generating models diagram' if @options.verbose
+    warn 'Generating models diagram' if @options.verbose
     get_files.each do |f|
       begin
         process_class extract_class_name(f).constantize
-      rescue Exception
-        STDERR.puts "Warning: exception #{$ERROR_INFO} raised while trying to load model class #{f}"
+      rescue NoMethodError
+        warn "Warning: exception #{$ERROR_INFO} raised while trying to load model class #{f}"
       end
     end
   end
 
   def get_files(prefix = '')
-    files = !@options.specify.empty? ? Dir.glob(@options.specify) : Dir.glob(prefix + 'app/models/**/*.rb')
+    files = !@options.specify.empty? ? Dir.glob(@options.specify) : Dir.glob("#{prefix}app/models/**/*.rb")
     files += Dir.glob('vendor/plugins/**/app/models/*.rb') if @options.plugins_models
-    files -= Dir.glob(prefix + 'app/models/concerns/**/*.rb') unless @options.include_concerns
+    files -= Dir.glob("#{prefix}app/models/concerns/**/*.rb") unless @options.include_concerns
     files += engine_files if @options.engine_models
     files -= Dir.glob(@options.exclude)
     files
@@ -41,17 +41,19 @@ class ModelsDiagram < AppDiagram
   end
 
   def extract_class_name(filename)
-    filename_was, class_name = filename, nil
+    filename_was = filename
+    class_name = nil
 
     filename = "app/models/#{filename.split('app/models')[1]}"
 
     while filename.split('/').length > 2
       begin
-        class_name = filename.match(/.*\/models\/(.*).rb$/)[1].camelize
+        class_name = filename.match(%r{.*/models/(.*).rb$})[1]
+        class_name = class_name.camelize
+        # class_name = class_name.from 2 if class_name.start_with? '::'
         class_name.constantize
-
         break
-      rescue Exception
+      rescue NoMethodError
         class_name = nil
         filename_end = filename.split('/')[2..-1]
         filename_end.shift
@@ -60,15 +62,16 @@ class ModelsDiagram < AppDiagram
     end
 
     if class_name.nil?
-      filename_was.match(/.*\/models\/(.*).rb$/)[1].camelize
+      filename_was.match(%r{.*/models/(.*).rb$})[1].camelize
     else
+      warn class_name
       class_name
     end
   end
 
   # Process a model class
   def process_class(current_class)
-    STDERR.puts "Processing #{current_class}" if @options.verbose
+    warn "Processing #{current_class}" if @options.verbose
 
     generated =
       if defined?(CouchRest::Model::Base) && current_class.new.is_a?(CouchRest::Model::Base)
@@ -85,13 +88,14 @@ class ModelsDiagram < AppDiagram
         process_basic_module(current_class)
       end
 
+    warn generated
     if @options.inheritance && generated && include_inheritance?(current_class)
       @graph.add_edge ['is-a', current_class.superclass.name, current_class.name]
     end
-  end # process_class
+  end
 
   def include_inheritance?(current_class)
-    STDERR.puts current_class.superclass if @options.verbose
+    warn current_class.superclass if @options.verbose
     (defined?(ActiveRecord::Base) ? current_class.superclass != ActiveRecord::Base : true) &&
       (defined?(CouchRest::Model::Base) ? current_class.superclass != CouchRest::Model::Base : true) &&
       (current_class.superclass != Object)
@@ -121,16 +125,16 @@ class ModelsDiagram < AppDiagram
       if @options.hide_magic
         # From patch #13351
         # http://wiki.rubyonrails.org/rails/pages/MagicFieldNames
-        magic_fields = %w(created_at created_on updated_at updated_on lock_version type id position parent_id lft rgt quote template)
-        magic_fields << current_class.table_name + '_count' if current_class.respond_to? 'table_name'
-        content_columns = current_class.content_columns.select { |c| !magic_fields.include? c.name }
+        magic_fields = %w[created_at created_on updated_at updated_on lock_version type id position parent_id lft rgt quote template]
+        magic_fields << "#{current_class.table_name}_count" if current_class.respond_to? 'table_name'
+        content_columns = current_class.content_columns.reject { |c| magic_fields.include? c.name }
       else
         content_columns = current_class.columns
       end
 
       content_columns.each do |a|
         content_column = a.name
-        content_column += ' :' + a.sql_type.to_s unless @options.hide_types
+        content_column += " :#{a.sql_type.to_s}" unless @options.hide_types
         node_attribs << content_column
       end
     end
@@ -141,7 +145,7 @@ class ModelsDiagram < AppDiagram
     if @options.inheritance && ! @options.transitive
       superclass_associations = current_class.superclass.reflect_on_all_associations
 
-      associations = associations.select { |a| !superclass_associations.include? a }
+      associations = associations.reject { |a| superclass_associations.include? a }
       # This doesn't works!
       # associations -= current_class.superclass.reflect_on_all_associations
     end
@@ -167,13 +171,13 @@ class ModelsDiagram < AppDiagram
         # From patch #13351
         # http://wiki.rubyonrails.org/rails/pages/MagicFieldNames
         magic_fields =
-          %w(created_at created_on updated_at updated_on lock_version _type _id position parent_id lft rgt quote template)
-        props = props.select { |c| !magic_fields.include?(c.name.to_s) }
+          %w[created_at created_on updated_at updated_on lock_version _type _id position parent_id lft rgt quote template]
+        props = props.reject { |c| magic_fields.include?(c.name.to_s) }
       end
 
       props.each do |a|
         prop = a.name.to_s
-        prop += ' :' + a.class.name.split('::').last unless @options.hide_types
+        prop += " :#{a.class.name.split('::').last}" unless @options.hide_types
         node_attribs << prop
       end
     end
@@ -205,8 +209,8 @@ class ModelsDiagram < AppDiagram
       if @options.hide_magic
         # From patch #13351
         # http://wiki.rubyonrails.org/rails/pages/MagicFieldNames
-        magic_fields = %w(created_at created_on updated_at updated_on lock_version _type _id position parent_id lft rgt quote template)
-        content_columns = content_columns.select { |c| !magic_fields.include?(c.name) }
+        magic_fields = %w[created_at created_on updated_at updated_on lock_version _type _id position parent_id lft rgt quote template]
+        content_columns = content_columns.reject { |c| magic_fields.include?(c.name) }
       end
 
       content_columns.each do |a|
@@ -251,8 +255,8 @@ class ModelsDiagram < AppDiagram
       content_columns = current_class.properties
 
       if @options.hide_magic
-        magic_fields = %w(created_at updated_at type _id _rev)
-        content_columns = content_columns.select { |c| !magic_fields.include?(c.name) }
+        magic_fields = %w[created_at updated_at type _id _rev]
+        content_columns = content_columns.reject { |c| magic_fields.include?(c.name) }
       end
 
       content_columns.each do |a|
@@ -269,11 +273,10 @@ class ModelsDiagram < AppDiagram
 
   # Process a model association
   def process_association(class_name, assoc)
-    STDERR.puts "- Processing model association #{assoc.name}" if @options.verbose
-
+    warn "- Processing model association #{assoc.name}" if @options.verbose
     # Skip "belongs_to" associations
-    macro = assoc.macro.to_s
-    return if %w(belongs_to referenced_in).include?(macro) && !@options.show_belongs_to
+    macro = assoc.methods.to_s
+    return if %w[belongs_to referenced_in].include?(macro) && !@options.show_belongs_to
 
     # Skip "through" associations
     through = assoc.options.include?(:through)
@@ -299,29 +302,30 @@ class ModelsDiagram < AppDiagram
     if class_name.include?('::') && !assoc_class_name.include?('::')
       assoc_class_name = class_name.split('::')[0..-2].push(assoc_class_name).join('::')
     end
-    assoc_class_name.gsub!(/^::/, '')
+    assoc_class_name = assoc_class_name.gsub(/^::/, '')
 
-    if %w(has_one references_one embeds_one).include?(macro)
+    if %w[has_one references_one embeds_one].include?(macro)
       assoc_type = 'one-one'
     elsif macro == 'has_many' && (!assoc.options[:through]) ||
-          %w(references_many embeds_many).include?(macro)
+          %w[references_many embeds_many].include?(macro)
       assoc_type = 'one-many'
     elsif macro == 'belongs_to'
       assoc_type = 'belongs-to'
     else # habtm or has_many, :through
       # Add FAKE associations too in order to understand mistakes
       return if @habtm.include? [assoc_class_name, class_name, assoc_name]
+
       assoc_type = 'many-many'
       @habtm << [class_name, assoc_class_name, assoc_name]
     end
     # from patch #12384
     # @graph.add_edge [assoc_type, class_name, assoc.class_name, assoc_name]
     @graph.add_edge [assoc_type, class_name, assoc_class_name, assoc_name]
-  end # process_association
+  end
 
   # Process a DataMapper relationship
   def process_datamapper_relationship(class_name, relation)
-    STDERR.puts "- Processing DataMapper model relationship #{relation.name}" if @options.verbose
+    warn "- Processing DataMapper model relationship #{relation.name}" if @options.verbose
 
     # Skip "belongs_to" relationships
     dm_type = relation.class.to_s.split('::')[-2]
@@ -354,4 +358,4 @@ class ModelsDiagram < AppDiagram
 
     @graph.add_edge [rel_type, class_name, assoc_class_name, assoc_name]
   end
-end # class ModelsDiagram
+end

data/lib/prailroady/options_struct.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # PrailRoady - RoR diagrams generator
 # http://railroad.rubyforge.org
 #
@@ -40,7 +41,7 @@ class OptionsStruct < OpenStruct
                      config_file: 'config/environment',
                      app_name: 'orailroady', app_human_name: 'PrailRoady', app_version: '', copyright: '' }
     super(init_options.merge(args))
-  end # initialize
+  end
 
   def parse(args)
     @opt_parser = OptionParser.new do |opts|
@@ -138,11 +139,11 @@ class OptionsStruct < OpenStruct
       opts.separator ''
       opts.separator 'Other options:'
       opts.on('-h', '--help', 'Show this message') do
-        STDOUT.print "#{opts}\n"
+        $stdout.print "#{opts}\n"
         exit
       end
       opts.on('--version', 'Show version and copyright') do
-        STDOUT.print "#{app_human_name} version #{app_version}\n\n" \
+        $stdout.print "#{app_human_name} version #{app_version}\n\n" \
                      "#{copyright}\nThis is free software; see the source " \
                      "for copying conditions.\n\n"
         exit
@@ -154,7 +155,7 @@ class OptionsStruct < OpenStruct
       opts.separator 'Commands (you must supply one of these):'
       opts.on('-M', '--models', 'Generate models diagram') do |_c|
         if command != ''
-          STDERR.print "Error: Can only generate one diagram type\n\n"
+          $stderr.print "Error: Can only generate one diagram type\n\n"
           exit 1
         else
           self.command = 'models'
@@ -162,16 +163,16 @@ class OptionsStruct < OpenStruct
       end
       opts.on('-C', '--controllers', 'Generate controllers diagram') do |_c|
         if command != ''
-          STDERR.print "Error: Can only generate one diagram type\n\n"
+          $stderr.print "Error: Can only generate one diagram type\n\n"
           exit 1
         else
           self.command = 'controllers'
         end
       end
       # From Ana Nelson's patch
-      opts.on('-A', '--aasm', "Generate \"acts as state machine\" diagram") do |_c|
+      opts.on('-A', '--aasm', 'Generate "acts as state machine" diagram') do |_c|
         if command == 'controllers'
-          STDERR.print "Error: Can only generate one diagram type\n\n"
+          $stderr.print "Error: Can only generate one diagram type\n\n"
           exit 1
         else
           self.command = 'aasm'
@@ -180,7 +181,7 @@ class OptionsStruct < OpenStruct
       opts.separator ''
       opts.separator 'For bug reporting and additional information, please see:'
       opts.separator 'http://railroad.rubyforge.org/'
-    end # do
+    end
 
     begin
       @opt_parser.parse!(args)
@@ -193,12 +194,12 @@ class OptionsStruct < OpenStruct
     rescue OptionParser::MissingArgument
       option_error 'Missing argument'
     end
-  end  # parse
+  end
 
   private
 
   def option_error(msg)
-    STDERR.print "Error: #{msg}\n\n #{@opt_parser}\n"
+    $stderr.print "Error: #{msg}\n\n #{@opt_parser}\n"
     exit 1
   end
-end  # class OptionsStruct
+end

data/lib/prailroady/version.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module PrailRoady
-  VERSION = '1.5.3'
+  VERSION = '1.6.0'
 end

data/lib/prailroady.rb

@@ -1,5 +1,13 @@
-%w(version options_struct models_diagram controllers_diagram aasm_diagram).each { |f| require "prailroady/#{f}" }
+# frozen_string_literal: true
 
-module PrailRoady
+require 'prailroady/version'
+require 'prailroady/options_struct'
+require 'prailroady/models_diagram'
+require 'prailroady/controllers_diagram'
+require 'prailroady/aasm_diagram'
+
+# This is the RailRoady module
+# TODO: documentation
+module RailRoady
   require 'prailroady/railtie' if defined?(Rails)
 end

data/prailroady.gemspec

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # coding: utf-8
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prailroady
 version: !ruby/object:Gem::Version
-  version: 1.5.3
+  version: 1.6.0
 platform: ruby
 authors:
 - SaKKo
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-07-01 00:00:00.000000000 Z
+date: 2022-03-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -118,7 +118,7 @@ homepage: http://github.com/sakko/prailroady
 licenses:
 - GPL-2.0
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -133,9 +133,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.8
-signing_key: 
+rubygems_version: 3.2.22
+signing_key:
 specification_version: 4
 summary: Ruby on Rails 3/4/5 model UML class diagram generator.
 test_files: