[WIP] Convert image links to shortened URLs

app/controllers/users_controller.rb

@@ -131,5 +131,4 @@ def allow_iframe_request
     # cannot-display-my-rails-4-app-in-iframe-even-if-x-frame-options-is-allowall
   end
 
-
 end

app/models/company.rb

@@ -177,6 +177,12 @@ def se_mailing_csv_str
       AddressExporter.se_mailing_csv_str( main_address )
   end
 
+  def get_or_create_short_h_brand_url(user)
+    found = self.short_h_brand_url
+    return found unless found.nil?
+    ShortenUrl.short(company_id: self.id)
+  end
+
 
   private
 

app/services/shorten_url.rb

@@ -0,0 +1,16 @@
+class ShortenUrl
+
+  SHORTEN_URL_LOG = 'log/tinyurl.log'
+
+  def self.short(url)
+    response = HTTParty.get("http://tinyurl.com/api-create.php?url=#{url}")
+    raise HTTParty::Error.new response if response.match? 'ERROR'
+    response
+  rescue HTTParty::Error => error
+    ActivityLogger.open(SHORTEN_URL_LOG, 'TINYURL_API', 'shortening url', false) do |log|
+      log.record('error', error.message)
+    end 
+    nil
+  end
+end
+

app/views/users/_company_h_brand.html.haml

@@ -35,7 +35,7 @@
         %br
         %br
         = t('users.show.use_this_image_link_html')
-        = company_h_brand_url(user, company_id: company.id)
+        = short_company_h_brand_url(user, company.id)
 
       :javascript
         window.onload = function(){

config/initializers/shorten_url.rb

@@ -0,0 +1 @@
+TINYURL_LOG = 'log/shorten_url.log'

db/migrate/20180713091110_add_short_brand_h_url_and_short_proof_of_membership_url_to_companies.rb

@@ -0,0 +1,6 @@
+class AddShortBrandHUrlAndShortProofOfMembershipUrlToCompanies < ActiveRecord::Migration[5.1]
+  def change
+    add_column :companies, :short_h_brand_url, :string
+    add_column :companies, :short_proof_of_membership_url, :string
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20180428103625) do
+ActiveRecord::Schema.define(version: 20180713091110) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -94,6 +94,9 @@
     t.datetime "updated_at", null: false
     t.text "description"
     t.string "dinkurs_company_id"
+    t.boolean "show_dinkurs_events"
+    t.string "short_h_brand_url"
+    t.string "short_proof_of_membership_url"
     t.index ["company_number"], name: "index_companies_on_company_number", unique: true
   end
 
@@ -240,9 +243,6 @@
   add_foreign_key "company_applications", "companies"
   add_foreign_key "company_applications", "shf_applications"
   add_foreign_key "events", "companies"
-  add_foreign_key "company_applications", "companies"
-  add_foreign_key "company_applications", "shf_applications"
-  add_foreign_key "events", "companies"
   add_foreign_key "payments", "companies"
   add_foreign_key "payments", "users"
   add_foreign_key "shf_applications", "member_app_waiting_reasons", column: "member_app_waiting_reasons_id"

db/structure.sql

@@ -239,7 +239,9 @@ CREATE TABLE public.companies (
     updated_at timestamp without time zone NOT NULL,
     description text,
     dinkurs_company_id character varying,
-    show_dinkurs_events boolean
+    show_dinkurs_events boolean,
+    short_h_brand_url character varying,
+    short_proof_of_membership_url character varying
 );
 
 
@@ -1318,6 +1320,7 @@ INSERT INTO "schema_migrations" (version) VALUES
 ('20180326103433'),
 ('20180328105100'),
 ('20180428103625'),
-('20180624155644');
+('20180624155644'),
+('20180713091110');
 
 

spec/factories/companies.rb

@@ -16,6 +16,8 @@
     phone_number '123123123'
     email 'thiscompany@example.com'
     website 'http://www.example.com'
+    short_h_brand_url nil
+    short_proof_of_membership_url nil
 
     transient do
       num_addresses 1

spec/models/company_spec.rb

@@ -5,6 +5,10 @@
 
 RSpec.describe Company, type: :model do
 
+  let(:with_short_urls) do
+    create(:company, short_h_brand_url: 'http://www.tinyurl.com/hbrand', short_proof_of_membership_url: 'http://tinyurl.com/memproof')
+  end
+
   let(:no_name) do
     create(:company, name: '', company_number: '2120000142')
   end
@@ -94,6 +98,8 @@
     it { is_expected.to have_db_column :description }
     it { is_expected.to have_db_column :dinkurs_company_id }
     it { is_expected.to have_db_column :show_dinkurs_events }
+    it { is_expected.to have_db_column :short_h_brand_url }
+    it { is_expected.to have_db_column :short_proof_of_membership_url }
   end
 
   describe 'Validations' do
@@ -697,4 +703,18 @@
     end
   end
 
+  describe '#get_or_create_short_h_brand_url', focus: true do
+    context 'there is already a shortened url in the table' do
+      it 'returns shortened url' do
+        expect(with_short_urls.get_or_create_short_h_brand_url(user)).to eq('http://www.tinyurl.com/hbrand')
+      end
+    end 
+    context 'there is no shortened url in the table and ShortenUrl.short is called' do
+      it 'saves the result if the result is not nil and returns shortened url' do
+        allow(ShortenUrl).to receive(:short).and_return('http://tinyurl.com/hbrand2')
+        expect(complete_co.get_or_create_short_h_brand_url(user)).to eq(ShortenUrl.short('a'))
+      end
+      it 'does not save anything if the result is nil and returns unshortened url'
+    end
+  end
 end

spec/services/shorten_url_spec.rb

@@ -0,0 +1,19 @@
+require 'rails_helper'
+
+describe ShortenUrl do
+  describe '.short' do
+    it 'creates shortened link' do
+      VCR.use_cassette('shorten_url/short') do
+        shortened_url = ShortenUrl.short 'http://sverigeshundforetagare.se/anvandare/2/proof_of_membership'
+        expect(shortened_url).to match 'tinyurl.com'
+      end
+    end
+    it 'if the service raises an error, returns nil and writes to the log' do
+      VCR.use_cassette('shorten_url/short') do 
+        expect(ActivityLogger).to receive(:open)
+        shortened_url = ShortenUrl.short '/'
+        expect(shortened_url).to eq nil
+      end
+    end
+  end
+end