BLS API GET conversion

app/lib/data_html_parser.rb

@@ -22,7 +22,7 @@ def get_fred_series(code, frequency = nil, aggregation_method = nil)
     data
   end
 
-  def get_bls_series_ONHOLD(series_id, _ = nil)
+  def get_bls_series(series_id, _ = nil)
     api_key = ENV['API_KEY_BLS'] || raise('No API key defined for BLS')
     thisyear = Date.today.year
     @url = 'https://api.bls.gov/publicAPI/v2/timeseries/data/%s?registration_key=%s&startyear=%d&endyear=%d' %
@@ -46,7 +46,7 @@ def get_bls_series_ONHOLD(series_id, _ = nil)
 
   ## Should be obsolete now, but let's keep it around a while longer just in case a problem arises
   ## with the new routine above
-  def get_bls_series(code, frequency = nil)
+  def get_bls_series_DELETEME(code, frequency = nil)
     @code = code
     @url = 'https://data.bls.gov/pdq/SurveyOutputServlet'
     @post_parameters = {

app/models/data_source.rb

@@ -243,7 +243,7 @@ def reload_source(clear_first = clear_before_load?)
         self.reload if presave_hook  ## it sucks to have to do this, but presave_hook might change something, that will end up saved below
         self.update!(update_props)
       end
-      Rails.logger.info { "Completed reload of definition #{id} for series <#{self.series}> [#{description}]" }
+      Rails.logger.info { "Completed reload of definition #{id} for series #{series.id} <#{series}> [#{description}]" }
       true
     end