From 3d2a0a060bc1080b7682685549f728934d5cb5f9 Mon Sep 17 00:00:00 2001
From: shawn-higgins1 <23224097+shawn-higgins1@users.noreply.github.com>
Date: Wed, 21 Aug 2019 14:23:00 -0400
Subject: [PATCH] GRN2-225, GRN2-227: Images sent by Office365 Institutional
 accounts break the rooms (#753)

* Validate profile image url

* Add rake task to migrate old office365 accounts
---
 app/helpers/application_helper.rb          |  8 ++++++
 app/views/shared/_header.html.erb          |  2 +-
 lib/tasks/migrate_old_office365_users.rake | 33 ++++++++++++++++++++++
 3 files changed, 42 insertions(+), 1 deletion(-)
 create mode 100644 lib/tasks/migrate_old_office365_users.rake

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index e1e901c2..42bcce1d 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -17,6 +17,7 @@
 # with BigBlueButton; if not, see <http://www.gnu.org/licenses/>.
 
 require 'bbb_api'
+require 'uri'
 require 'i18n/language/mapping'
 
 module ApplicationHelper
@@ -139,4 +140,11 @@ module ApplicationHelper
   def google_analytics_url
     "https://www.googletagmanager.com/gtag/js?id=#{ENV['GOOGLE_ANALYTICS_TRACKING_ID']}"
   end
+
+  def valid_url?(input)
+    uri = URI.parse(input)
+    !uri.host.nil?
+  rescue URI::InvalidURIError
+    false
+  end
 end
diff --git a/app/views/shared/_header.html.erb b/app/views/shared/_header.html.erb
index c8efbc60..069fa03a 100755
--- a/app/views/shared/_header.html.erb
+++ b/app/views/shared/_header.html.erb
@@ -44,7 +44,7 @@
 
           <div class="dropdown">
             <a href="#" class="nav-link pr-0" data-toggle="dropdown">
-              <% if current_user.image.blank? %>
+              <% if current_user.image.blank? || !valid_url?(current_user.image) %>
                 <span class="avatar"><%= current_user.name.first %></span>
               <% else %>
                 <span id="user-avatar" class="avatar d-none"><%= current_user.name.first %></span>
diff --git a/lib/tasks/migrate_old_office365_users.rake b/lib/tasks/migrate_old_office365_users.rake
new file mode 100644
index 00000000..aaf3cdbc
--- /dev/null
+++ b/lib/tasks/migrate_old_office365_users.rake
@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+namespace :office365 do
+  desc "Migrates over old office365 users to new account"
+  task :migrate, [] => :environment do |_task, _args|
+    old_office_users = User.where(provider: "microsoft_office365")
+
+    old_office_users.each do |old_user|
+      new_social_uid = if old_user.email.match("^outlook_[0-9a-zA-Z]+@outlook.com$")
+        old_user.email.last(old_user.email.length - 8).split('@')[0]
+      else
+        old_user.social_uid.split('@')[0]
+      end
+
+      new_user = User.where(provider: "office365", social_uid: new_social_uid).first
+
+      if new_user.nil?
+        old_user.provider = "office365"
+        old_user.social_uid = new_social_uid
+        old_user.save!
+      else
+        old_main_room = old_user.main_room
+        old_main_room.name = "Old " + old_main_room.name
+        old_main_room.save!
+
+        new_user.rooms << old_user.rooms
+        new_user.role_ids = new_user.role_ids | old_user.role_ids
+        new_user.save!
+        old_user.delete
+      end
+    end
+  end
+end