app/commands/decidim/system/update_oauth_application.rb in decidim-system-0.24.3 vs app/commands/decidim/system/update_oauth_application.rb in decidim-system-0.25.0.rc1

@@ -19,21 +19,28 @@
         return broadcast(:invalid) unless @form.valid?
 
         @application = Decidim.traceability.update!(
           @application,
           @user,
-          name: @form.name,
-          decidim_organization_id: @form.decidim_organization_id,
-          organization_name: @form.organization_name,
-          organization_url: @form.organization_url,
-          organization_logo: @form.organization_logo,
-          redirect_uri: @form.redirect_uri
+          **oauth_application_attributes
         )
 
         broadcast(:ok, @application)
       rescue ActiveRecord::RecordInvalid
         @form.errors.add(:organization_logo, @application.errors[:organization_logo]) if @application.errors.include? :organization_logo
         broadcast(:invalid)
+      end
+
+      def oauth_application_attributes
+        {
+          name: @form.name,
+          decidim_organization_id: @form.decidim_organization_id,
+          organization_name: @form.organization_name,
+          organization_url: @form.organization_url,
+          redirect_uri: @form.redirect_uri
+        }.tap do |attrs|
+          attrs[:organization_logo] = @form.organization_logo if @form.organization_logo.present?
+        end
       end
     end
   end
 end