Commits on Mar 3, 2017

1. Use the already included module instead of reopening classes. …

   Related to ManageIQ#14128
   
   We should either use RssFeed.class_eval to reopen the class to force
   rails to load the RssFeed model first or the technique in this commit,
   which is to just include the module in RssFeed.  Note, this module was
   already being included so there was no reason to reopen the class also.
   
   The existing layout of re-opening the RssFeed with different class
   definitions:
   
   rss_feed/import_export.rb:
   class RssFeed
   
   rss_feed.rb:
   class RssFeed < ApplicationRecord
   
   can cause `TypeError: superclass mismatch for class RssFeed` if
   the import_export.rb is loaded first.

   

   jrafanie committed Mar 3, 2017
   Configuration menu

   • View commit details
   • Copy full SHA for 0229960
   • Browse repository at this point

   Copy the full SHA

   0229960 View commit details

   Browse the repository at this point in the history