Add website url to user

Jerome Dalbert
1 parent dba98240
Showing 14 changed files with 94 additions and 9 deletions Show diff stats
CHANGELOG
app/models/user.rb
app/views/admin/users/_form.html.haml
app/views/profiles/show.html.haml
app/views/users/_profile.html.haml
db/migrate/20140116231608_add_website_url_to_users.rb
db/schema.rb
doc/api/session.md
doc/api/users.md
features/profile/profile.feature
features/steps/profile/profile.rb
lib/api/entities.rb
lib/api/users.rb
spec/models/user_spec.rb
@@ -16,6 +16,7 @@ v 6.5.0
   - Use jquery timeago plugin
   - Fix 500 error for rdoc files
   - Ability to customize merge commit message (sponsored by Say Media)
+  - Add website url to user profile
 v6.4.3
   - Don't use unicorn worker killer if PhusionPassenger is defined
@@ -41,7 +41,8 @@
 #  confirmed_at           :datetime
 #  confirmation_sent_at   :datetime
 #  unconfirmed_email      :string(255)
-#  hide_no_ssh_key        :boolean          default(FALSE), not null
+#  hide_no_ssh_key        :boolean          default(FALSE)
+#  website_url            :string(255)      default(""), not null
 #
 require 'carrierwave/orm/activerecord'
@@ -52,7 +53,7 @@ class User &lt; ActiveRecord::Base
          :recoverable, :rememberable, :trackable, :validatable, :omniauthable, :confirmable, :registerable
   attr_accessible :email, :password, :password_confirmation, :remember_me, :bio, :name, :username,
-                  :skype, :linkedin, :twitter, :color_scheme_id, :theme_id, :force_random_password,
+                  :skype, :linkedin, :twitter, :website_url, :color_scheme_id, :theme_id, :force_random_password,
                   :extern_uid, :provider, :password_expires_at, :avatar, :hide_no_ssh_key,
                   as: [:default, :admin]
@@ -424,4 +425,14 @@ class User &lt; ActiveRecord::Base
       order('id DESC').limit(1000).
       update_all(updated_at: Time.now)
   end
+
+  def full_website_url
+    return "http://#{website_url}" if website_url !~ /^https?:\/\//
+
+    website_url
+  end
+
+  def short_website_url
+    website_url.gsub(/https?:\/\//, '')
+  end
 end
@@ -80,6 +80,9 @@
       .form-group
         = f.label :twitter, class: 'control-label'
         .col-sm-10= f.text_field :twitter, class: 'form-control'
+      .form-group
+        = f.label :website_url, class: 'control-label'
+        .col-sm-10= f.text_field :website_url, class: 'form-control'
     .form-actions
       - if @user.new_record?
@@ -47,6 +47,9 @@
         = f.label :twitter, class: "control-label"
         .col-sm-10= f.text_field :twitter, class: "form-control"
       .form-group
+        = f.label :website_url, class: "control-label"
+        .col-sm-10= f.text_field :website_url, class: "form-control"
+      .form-group
         = f.label :bio, class: "control-label"
         .col-sm-10
           = f.text_area :bio, rows: 6, class: "form-control", maxlength: 250
@@ -17,6 +17,10 @@
       %li
         %span.light Twitter:
         %strong= link_to user.twitter, "http://www.twitter.com/#{user.twitter}"
+    - unless user.website_url.blank?
+      %li
+        %span.light Website url:
+        %strong= link_to user.short_website_url, user.full_website_url
     - unless user.bio.blank?
       %li
         %span.light Bio:
@@ -0,0 +1,5 @@
+class AddWebsiteUrlToUsers < ActiveRecord::Migration
+  def change
+    add_column :users, :website_url, :string, {:null => false, :default => ''}
+  end
+end
@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
-ActiveRecord::Schema.define(version: 20131217102743) do
+ActiveRecord::Schema.define(version: 20140116231608) do
   create_table "broadcast_messages", force: true do |t|
     t.text     "message",    null: false
@@ -301,6 +301,7 @@ ActiveRecord::Schema.define(version: 20131217102743) do
     t.datetime "confirmation_sent_at"
     t.string   "unconfirmed_email"
     t.boolean  "hide_no_ssh_key",        default: false
+    t.string   "website_url",            default: "",    null: false
   end
   add_index "users", ["admin"], name: "index_users_on_admin", using: :btree
@@ -26,6 +26,7 @@ __You can login with both GitLab and LDAP credentials now__
   "skype": "",
   "linkedin": "",
   "twitter": "",
+  "website_url": "",
   "dark_scheme": false,
   "theme_id": 1,
   "is_admin": false,
@@ -20,6 +20,7 @@ GET /users
     "skype": "",
     "linkedin": "",
     "twitter": "",
+    "website_url": "",
     "extern_uid": "john.smith",
     "provider": "provider_name",
     "theme_id": 1,
@@ -38,6 +39,7 @@ GET /users
     "skype": "",
     "linkedin": "",
     "twitter": "",
+    "website_url": "",
     "extern_uid": "jack.smith",
     "provider": "provider_name",
     "theme_id": 1,
@@ -74,6 +76,7 @@ Parameters:
   "skype": "",
   "linkedin": "",
   "twitter": "",
+  "website_url": "",
   "extern_uid": "john.smith",
   "provider": "provider_name",
   "theme_id": 1,
@@ -102,6 +105,7 @@ Parameters:
 + `skype` (optional)            - Skype ID
 + `linkedin` (optional)         - Linkedin
 + `twitter` (optional)          - Twitter account
++ `website_url` (optional)      - Website url
 + `projects_limit` (optional)   - Number of projects user can create
 + `extern_uid` (optional)       - External UID
 + `provider` (optional)         - External provider name
@@ -127,6 +131,7 @@ Parameters:
 + `skype`                             - Skype ID
 + `linkedin`                          - Linkedin
 + `twitter`                           - Twitter account
++ `website_url`                       - Website url
 + `projects_limit`                    - Limit projects each user can create
 + `extern_uid`                        - External UID
 + `provider`                          - External provider name
@@ -174,6 +179,7 @@ GET /user
   "skype": "",
   "linkedin": "",
   "twitter": "",
+  "website_url": "",
   "theme_id": 1,
   "color_scheme_id": 2,
   "is_admin": false,
@@ -8,8 +8,8 @@ Feature: Profile
   Scenario: I edit profile
     Given I visit profile page
-    Then I change my contact info
-    And I should see new contact info
+    Then I change my profile info
+    And I should see new profile info
   Scenario: I change my password without old one
     Given I visit profile password page
@@ -6,18 +6,20 @@ class Profile &lt; Spinach::FeatureSteps
     page.should have_content "Profile settings"
   end
-  step 'I change my contact info' do
+  step 'I change my profile info' do
     fill_in "user_skype", with: "testskype"
     fill_in "user_linkedin", with: "testlinkedin"
     fill_in "user_twitter", with: "testtwitter"
+    fill_in "user_website_url", with: "testurl"
     click_button "Save changes"
     @user.reload
   end
-  step 'I should see new contact info' do
+  step 'I should see new profile info' do
     @user.skype.should == 'testskype'
     @user.linkedin.should == 'testlinkedin'
     @user.twitter.should == 'testtwitter'
+    @user.website_url.should == 'testurl'
   end
   step 'I change my avatar' do
 module API
   module Entities
     class User < Grape::Entity
-      expose :id, :username, :email, :name, :bio, :skype, :linkedin, :twitter,
+      expose :id, :username, :email, :name, :bio, :skype, :linkedin, :twitter, :website_url,
              :theme_id, :color_scheme_id, :state, :created_at, :extern_uid, :provider
       expose :is_admin?, as: :is_admin
       expose :can_create_group?, as: :can_create_group
@@ -36,6 +36,7 @@ module API
       #   skype                             - Skype ID
       #   linkedin                          - Linkedin
       #   twitter                           - Twitter account
+      #   website_url                       - Website url
       #   projects_limit                    - Number of projects user can create
       #   extern_uid                        - External authentication provider UID
       #   provider                          - External provider
@@ -67,6 +68,7 @@ module API
       #   skype                             - Skype ID
       #   linkedin                          - Linkedin
       #   twitter                           - Twitter account
+      #   website_url                       - Website url
       #   projects_limit                    - Limit projects each user can create
       #   extern_uid                        - External authentication provider UID
       #   provider                          - External provider
@@ -78,7 +80,7 @@ module API
       put ":id" do
         authenticated_as_admin!
-        attrs = attributes_for_keys [:email, :name, :password, :skype, :linkedin, :twitter, :projects_limit, :username, :extern_uid, :provider, :bio, :can_create_group, :admin]
+        attrs = attributes_for_keys [:email, :name, :password, :skype, :linkedin, :twitter, :website_url, :projects_limit, :username, :extern_uid, :provider, :bio, :can_create_group, :admin]
         user = User.find(params[:id])
         not_found!("User not found") unless user
@@ -41,6 +41,8 @@
 #  confirmed_at           :datetime
 #  confirmation_sent_at   :datetime
 #  unconfirmed_email      :string(255)
+#  hide_no_ssh_key        :boolean          default(FALSE)
+#  website_url            :string(255)      default(""), not null
 #
 require 'spec_helper'
@@ -293,4 +295,48 @@ describe User do
       user.avatar_type.should == ["only images allowed"]
     end
   end
+
+  describe '#full_website_url' do
+    let(:user) { create(:user) }
+
+    it 'begins with http if website url omits it' do
+      user.website_url = 'test.com'
+
+      expect(user.full_website_url).to eq 'http://test.com'
+    end
+
+    it 'begins with http if website url begins with http' do
+      user.website_url = 'http://test.com'
+
+      expect(user.full_website_url).to eq 'http://test.com'
+    end
+
+    it 'begins with https if website url begins with https' do
+      user.website_url = 'https://test.com'
+
+      expect(user.full_website_url).to eq 'https://test.com'
+    end
+  end
+
+  describe '#short_website_url' do
+    let(:user) { create(:user) }
+
+    it 'does not begin with http if website url omits it' do
+      user.website_url = 'test.com'
+
+      expect(user.short_website_url).to eq 'test.com'
+    end
+
+    it 'does not begin with http if website url begins with http' do
+      user.website_url = 'http://test.com'
+
+      expect(user.short_website_url).to eq 'test.com'
+    end
+
+    it 'does not begin with https if website url begins with https' do
+      user.website_url = 'https://test.com'
+
+      expect(user.short_website_url).to eq 'test.com'
+    end
+  end
 end
	@@ -16,6 +16,7 @@ v 6.5.0		@@ -16,6 +16,7 @@ v 6.5.0
16	- Use jquery timeago plugin	16	- Use jquery timeago plugin
17	- Fix 500 error for rdoc files	17	- Fix 500 error for rdoc files
18	- Ability to customize merge commit message (sponsored by Say Media)	18	- Ability to customize merge commit message (sponsored by Say Media)
		19	+ - Add website url to user profile
19		20
20	v6.4.3	21	v6.4.3
21	- Don't use unicorn worker killer if PhusionPassenger is defined	22	- Don't use unicorn worker killer if PhusionPassenger is defined
@@ -0,0 +1,5 @@		@@ -0,0 +1,5 @@
	1	+class AddWebsiteUrlToUsers < ActiveRecord::Migration
	2	+ def change
	3	+ add_column :users, :website_url, :string, {:null => false, :default => ''}
	4	+ end
	5	+end
1	module API	1	module API
2	module Entities	2	module Entities
3	class User < Grape::Entity	3	class User < Grape::Entity
4	- expose :id, :username, :email, :name, :bio, :skype, :linkedin, :twitter,	4	+ expose :id, :username, :email, :name, :bio, :skype, :linkedin, :twitter, :website_url,
5	:theme_id, :color_scheme_id, :state, :created_at, :extern_uid, :provider	5	:theme_id, :color_scheme_id, :state, :created_at, :extern_uid, :provider
6	expose :is_admin?, as: :is_admin	6	expose :is_admin?, as: :is_admin
7	expose :can_create_group?, as: :can_create_group	7	expose :can_create_group?, as: :can_create_group
	@@ -41,6 +41,8 @@		@@ -41,6 +41,8 @@
41	# confirmed_at :datetime	41	# confirmed_at :datetime
42	# confirmation_sent_at :datetime	42	# confirmation_sent_at :datetime
43	# unconfirmed_email :string(255)	43	# unconfirmed_email :string(255)
		44	+# hide_no_ssh_key :boolean default(FALSE)
		45	+# website_url :string(255) default(""), not null
44	#	46	#
45		47
46	require 'spec_helper'	48	require 'spec_helper'
	@@ -293,4 +295,48 @@ describe User do		@@ -293,4 +295,48 @@ describe User do
293	user.avatar_type.should == ["only images allowed"]	295	user.avatar_type.should == ["only images allowed"]
294	end	296	end
295	end	297	end
		298	+
		299	+ describe '#full_website_url' do
		300	+ let(:user) { create(:user) }
		301	+
		302	+ it 'begins with http if website url omits it' do
		303	+ user.website_url = 'test.com'
		304	+
		305	+ expect(user.full_website_url).to eq 'http://test.com'
		306	+ end
		307	+
		308	+ it 'begins with http if website url begins with http' do
		309	+ user.website_url = 'http://test.com'
		310	+
		311	+ expect(user.full_website_url).to eq 'http://test.com'
		312	+ end
		313	+
		314	+ it 'begins with https if website url begins with https' do
		315	+ user.website_url = 'https://test.com'
		316	+
		317	+ expect(user.full_website_url).to eq 'https://test.com'
		318	+ end
		319	+ end
		320	+
		321	+ describe '#short_website_url' do
		322	+ let(:user) { create(:user) }
		323	+
		324	+ it 'does not begin with http if website url omits it' do
		325	+ user.website_url = 'test.com'
		326	+
		327	+ expect(user.short_website_url).to eq 'test.com'
		328	+ end
		329	+
		330	+ it 'does not begin with http if website url begins with http' do
		331	+ user.website_url = 'http://test.com'
		332	+
		333	+ expect(user.short_website_url).to eq 'test.com'
		334	+ end
		335	+
		336	+ it 'does not begin with https if website url begins with https' do
		337	+ user.website_url = 'https://test.com'
		338	+
		339	+ expect(user.short_website_url).to eq 'test.com'
		340	+ end
		341	+ end
296	end	342	end