mno-enterprise-api 3.0.2 → 3.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2d433295f9c87eef1d54931eb24268c0d884dd7a
-  data.tar.gz: 4153fae7373b8991c6cd74d2e77a8666c06c5de3
+  metadata.gz: c2d4ffdd0869f327305258d29469731d30b31d5f
+  data.tar.gz: 3c5a02ab3f7fa05c6ea4a5ac80f40d2c89b46d9e
 SHA512:
-  metadata.gz: 8063295c92c5ec525ba18177120644ec0daf04609716a697c8dc63c9540018801adb9f18f3c171f357c503d866d4a9cedd297b77b5a63873535d1ff2f2696cd2
-  data.tar.gz: 98a9789331a4d92dffcbad5737f1a24e67413d3873093ac8ed4578902ddcab8f5753cf8e029179384c21699e8a524ac1ce30b53362cfff5021bcee0baac95db0
+  metadata.gz: 6ba45fee7bba517d7600815bb3f71933a3b93a31283f26cb2891a95f0cee75b118714444d8fa36c9aa45031f2ef3dbdd207143b08d80aa0f96b2518c42fee164
+  data.tar.gz: 32ac403e68f0a2e15d000a8b9cca4323c209b8c1d29ee07663f31ef7fea5c4309afad30196a081a490ac35e05fe2b82838642ccb72fe470d786a7a3c076653ac

data/app/controllers/mno_enterprise/jpi/v1/impac/widgets_controller.rb

@@ -1,63 +1,5 @@
 module MnoEnterprise
   class Jpi::V1::Impac::WidgetsController < Jpi::V1::BaseResourceController
-    respond_to :json
-    
-    # POST /mnoe/jpi/v1/impac/dashboards/:id/widgets
-    #  -> POST /api/mnoe/v1/dashboards/:id/widgets
-    def create
-      if widgets
-        if @widget = widgets.create(format_attrs(['widget_category','metadata']))
-          MnoEnterprise::EventLogger.info('widget_create', current_user.id, 'Widget Creation', nil, @widget)
-          @nocontent = true # no data fetch from Connec!
-          render 'show'
-        else
-          render json: @widget.errors, status: :bad_request
-        end
-      else
-        render json: { errors: "Dashboard id #{params[:id]} doesn't exist" }, status: :not_found
-      end
-    end
-
-    # PUT /mnoe/jpi/v1/impac/widgets/:id
-    def update
-      if widget.update(format_attrs(['name','metadata']))
-        @nocontent = !params['metadata']
-        render 'show'
-      else
-        render json: @widget.errors, status: :bad_request
-      end
-    end
-    
-    # DELETE /mnoe/jpi/v1/impac/dashboards/1
-    def destroy
-      if widget.destroy
-        MnoEnterprise::EventLogger.info('widget_delete', current_user.id, 'Widget Deletion', nil, widget)
-        head status: :ok
-      else
-        render json: 'Unable to destroy widget', status: :bad_request
-      end
-    end
-    
-    
-    #=================================================
-    # Private methods
-    #=================================================
-    private
-
-      def widget
-        @widget ||= MnoEnterprise::Impac::Widget.find(params[:id])
-      end
-
-      def widgets
-        @widgets ||= MnoEnterprise::Impac::Dashboard.find(params[:dashboard_id]).widgets
-      end
-
-      def format_attrs(whitelist)
-        attrs = (params[:widget] || {}).select { |k,v| whitelist.include?(k.to_s) }
-        attrs['settings'] = widget ? widget.settings || {} : {}
-        attrs['settings'].merge!(attrs['metadata']) if attrs['metadata']
-        attrs.except!('metadata')
-      end
-
+    include MnoEnterprise::Concerns::Controllers::Jpi::V1::Impac::WidgetsController
   end
 end

data/lib/mno_enterprise/concerns/controllers/jpi/v1/impac/widgets_controller.rb

@@ -0,0 +1,72 @@
+module MnoEnterprise::Concerns::Controllers::Jpi::V1::Impac::WidgetsController
+  extend ActiveSupport::Concern
+
+  #==================================================================
+  # Included methods
+  #==================================================================
+  # 'included do' causes the included code to be evaluated in the
+  # context where it is included rather than being executed in the module's context
+  included do
+    respond_to :json
+  end
+
+  #==================================================================
+  # Instance methods
+  #==================================================================
+  # POST /mnoe/jpi/v1/impac/dashboards/:id/widgets
+  #  -> POST /api/mnoe/v1/dashboards/:id/widgets
+  def create
+    if widgets
+      if @widget = widgets.create(format_attrs(['widget_category','metadata']))
+        MnoEnterprise::EventLogger.info('widget_create', current_user.id, 'Widget Creation', nil, @widget)
+        @nocontent = true # no data fetch from Connec!
+        render 'show'
+      else
+        render json: @widget.errors, status: :bad_request
+      end
+    else
+      render json: { errors: "Dashboard id #{params[:id]} doesn't exist" }, status: :not_found
+    end
+  end
+
+  # PUT /mnoe/jpi/v1/impac/widgets/:id
+  def update
+    if widget.update(format_attrs(['name','metadata']))
+      @nocontent = !params['metadata']
+      render 'show'
+    else
+      render json: @widget.errors, status: :bad_request
+    end
+  end
+  
+  # DELETE /mnoe/jpi/v1/impac/dashboards/1
+  def destroy
+    if widget.destroy
+      MnoEnterprise::EventLogger.info('widget_delete', current_user.id, 'Widget Deletion', nil, widget)
+      head status: :ok
+    else
+      render json: 'Unable to destroy widget', status: :bad_request
+    end
+  end
+  
+  
+  #=================================================
+  # Private methods
+  #=================================================
+  private
+
+    def widget
+      @widget ||= MnoEnterprise::Impac::Widget.find(params[:id])
+    end
+
+    def widgets
+      @widgets ||= MnoEnterprise::Impac::Dashboard.find(params[:dashboard_id]).widgets
+    end
+
+    def format_attrs(whitelist)
+      attrs = (params[:widget] || {}).select { |k,v| whitelist.include?(k.to_s) }
+      attrs['settings'] = widget ? widget.settings || {} : {}
+      attrs['settings'].merge!(attrs['metadata']) if attrs['metadata']
+      attrs.except!('metadata')
+    end
+end

data/lib/mno_enterprise/concerns/controllers/org_invites_controller.rb

@@ -29,7 +29,7 @@ module MnoEnterprise::Concerns::Controllers::OrgInvitesController
     redirect_path = mnoe_home_path
 
     if @org_invite && !@org_invite.expired? && @org_invite.accept!(current_user)
-      redirect_path += "#/?dhbRefId=#{ @org_invite.organization.id}"
+      redirect_path = add_param_to_fragment(redirect_path.to_s, 'dhbRefId', @org_invite.organization.id)
       message = { notice: "You are now part of #{@org_invite.organization.name}" }
       yield(:success, @org_invite) if block_given?
     elsif @org_invite && @org_invite.expired?

data/lib/mno_enterprise/concerns/mailers/system_notification_mailer.rb

@@ -128,7 +128,8 @@ module MnoEnterprise::Concerns::Mailers::SystemNotificationMailer
   protected
 
   def recipient(record, new_user = false)
-    hash = { email: record.email }
+    # Org Invite for unconfirmed users will have the email in #unconfirmed_email
+    hash = { email: record.email || record.unconfirmed_email }
     hash[:name] = "#{record.name} #{record.surname}".strip unless new_user
     hash
   end

data/spec/controllers/mno_enterprise/org_invites_controller_spec.rb

@@ -8,7 +8,7 @@ module MnoEnterprise
   describe OrgInvitesController, type: :controller do
     render_views
     routes { MnoEnterprise::Engine.routes }
-  
+
     let(:user) { build(:user) }
     let(:invite) { build(:org_invite, user: user) }
     let(:token) { invite.token }
@@ -25,7 +25,7 @@ module MnoEnterprise
     describe "GET #show" do
       subject { get :show, id: invite.id, token: token}
 
-      let(:success_fragment) { "#/?dhbRefId=#{invite.organization.id}&#{URI.encode_www_form([['flash', {msg: "You are now part of #{invite.organization.name}", type: :success}.to_json]])}" }
+      let(:success_fragment) { "#?dhbRefId=#{invite.organization.id}&#{URI.encode_www_form([['flash', {msg: "You are now part of #{invite.organization.name}", type: :success}.to_json]])}" }
       let(:expired_fragment) { "#?#{URI.encode_www_form([['flash', {msg: "It looks like this invite has expired. Please ask your company administrator to resend the invite.", type: :error}.to_json]])}" }
       let(:invalid_fragment) { "#?#{URI.encode_www_form([['flash', {msg: "Unfortunately, this invite does not seem to be valid.", type: :error}.to_json]])}" }
 

data/spec/controllers/mno_enterprise/webhook/o_auth_controller_spec.rb

@@ -33,7 +33,7 @@ module MnoEnterprise
       it_behaves_like 'a user protected resource'
 
       it { subject; expect(response).to be_success }
-      it { subject; expect(assigns(:redirect_to)).to eq(redirect_url) }
+      it { Timecop.freeze { subject; expect(assigns(:redirect_to)).to eq(redirect_url) } }
 
       Webhook::OAuthController::PROVIDERS_WITH_OPTIONS.each do |provider|
         describe "#{provider.capitalize} provider" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mno-enterprise-api
 version: !ruby/object:Gem::Version
-  version: 3.0.2
+  version: 3.0.3
 platform: ruby
 authors:
 - Arnaud Lachaume
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-12 00:00:00.000000000 Z
+date: 2016-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mno-enterprise-core
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.0.2
+        version: 3.0.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.0.2
+        version: 3.0.3
 - !ruby/object:Gem::Dependency
   name: jbuilder
   requirement: !ruby/object:Gem::Requirement
@@ -254,6 +254,7 @@ files:
 - lib/mno_enterprise/concerns/controllers/jpi/v1/current_users_controller.rb
 - lib/mno_enterprise/concerns/controllers/jpi/v1/deletion_requests_controller.rb
 - lib/mno_enterprise/concerns/controllers/jpi/v1/impac/dashboards_controller.rb
+- lib/mno_enterprise/concerns/controllers/jpi/v1/impac/widgets_controller.rb
 - lib/mno_enterprise/concerns/controllers/jpi/v1/organizations_controller.rb
 - lib/mno_enterprise/concerns/controllers/org_invites_controller.rb
 - lib/mno_enterprise/concerns/controllers/pages_controller.rb