decidim-core 0.9.3 → 0.10.0

data/app/models/decidim/newsletter.rb

@@ -3,12 +3,19 @@
 module Decidim
   # This model holds all the data needed to send a newsletter.
   class Newsletter < ApplicationRecord
+    include Decidim::Traceable
+    include Decidim::Loggable
+
     belongs_to :author, class_name: "User"
     belongs_to :organization
 
     validates :subject, :body, presence: true
     validate :author_belongs_to_organization
 
+    def self.log_presenter_class_for(_log)
+      Decidim::AdminLog::NewsletterPresenter
+    end
+
     # Returns true if this newsletter was already sent.
     #
     # Returns a Boolean.

data/app/models/decidim/organization.rb

@@ -5,11 +5,16 @@ module Decidim
   # installation we can find many organizations and each of them can start
   # their own participatory processes.
   class Organization < ApplicationRecord
+    include Decidim::Traceable
+    include Decidim::Loggable
+
     SOCIAL_HANDLERS = [:twitter, :facebook, :instagram, :youtube, :github].freeze
 
     has_many :static_pages, foreign_key: "decidim_organization_id", class_name: "Decidim::StaticPage", inverse_of: :organization, dependent: :destroy
     has_many :scopes, -> { order(name: :asc) }, foreign_key: "decidim_organization_id", class_name: "Decidim::Scope", inverse_of: :organization
     has_many :scope_types, -> { order(name: :asc) }, foreign_key: "decidim_organization_id", class_name: "Decidim::ScopeType", inverse_of: :organization
+    has_many :areas, -> { order(name: :asc) }, foreign_key: "decidim_organization_id", class_name: "Decidim::Area", inverse_of: :organization
+    has_many :area_types, -> { order(name: :asc) }, foreign_key: "decidim_organization_id", class_name: "Decidim::AreaType", inverse_of: :organization
     has_many :admins, -> { where(admin: true) }, foreign_key: "decidim_organization_id", class_name: "Decidim::User"
     has_many :users_with_any_role, -> { where.not(roles: []) }, foreign_key: "decidim_organization_id", class_name: "Decidim::User"
     has_many :users, foreign_key: "decidim_organization_id", class_name: "Decidim::User", dependent: :destroy
@@ -25,6 +30,10 @@ module Decidim
     mount_uploader :favicon, Decidim::OrganizationFaviconUploader
     mount_uploader :highlighted_content_banner_image, ImageUploader
 
+    def self.log_presenter_class_for(_log)
+      Decidim::AdminLog::OrganizationPresenter
+    end
+
     # Returns top level scopes for this organization.
     #
     # Returns an ActiveRecord::Relation.

data/app/models/decidim/participatory_process_user_role.rb

@@ -4,13 +4,22 @@ module Decidim
   # Defines a relation between a user and a participatory process, and what
   # kind of relation does the user has.
   class ParticipatoryProcessUserRole < ApplicationRecord
+    include Traceable
+    include Loggable
+
     belongs_to :user, foreign_key: "decidim_user_id", class_name: "Decidim::User", optional: true
     belongs_to :participatory_process, foreign_key: "decidim_participatory_process_id", class_name: "Decidim::ParticipatoryProcess", optional: true
 
+    alias participatory_space participatory_process
+
     ROLES = %w(admin collaborator moderator).freeze
     validates :role, inclusion: { in: ROLES }, uniqueness: { scope: [:user, :participatory_process] }
     validate :user_and_participatory_process_same_organization
 
+    def self.log_presenter_class_for(_log)
+      Decidim::ParticipatoryProcesses::AdminLog::ParticipatoryProcessUserRolePresenter
+    end
+
     private
 
     # Private: check if the process and the user have the same organization

data/app/models/decidim/resource_link.rb

@@ -13,8 +13,8 @@ module Decidim
     belongs_to :to, polymorphic: true
 
     validates :name, presence: true, uniqueness: { scope: [:from, :to] }
-
     validate :same_organization
+    validate :same_participatory_space
 
     private
 
@@ -24,5 +24,12 @@ module Decidim
       errors.add(:from, :invalid)
       errors.add(:to, :invalid)
     end
+
+    def same_participatory_space
+      return if from.try(:participatory_space) == to.try(:participatory_space)
+
+      errors.add(:from, :invalid)
+      errors.add(:to, :invalid)
+    end
   end
 end

data/app/models/decidim/scope.rb

@@ -5,6 +5,9 @@ module Decidim
   # the scope of a participatory process.
   # (i.e. does it affect the whole city or just a district?)
   class Scope < ApplicationRecord
+    include Decidim::Traceable
+    include Decidim::Loggable
+
     belongs_to :organization,
                foreign_key: "decidim_organization_id",
                class_name: "Decidim::Organization",
@@ -43,6 +46,10 @@ module Decidim
       where parent_id: nil
     end
 
+    def self.log_presenter_class_for(_log)
+      Decidim::AdminLog::ScopePresenter
+    end
+
     def descendants
       organization.scopes.where("? = ANY(decidim_scopes.part_of)", id)
     end

data/app/models/decidim/static_page.rb

@@ -8,6 +8,9 @@ module Decidim
   # Pages with a default slug cannot be destroyed and its slug cannot be
   # modified.
   class StaticPage < ApplicationRecord
+    include Decidim::Traceable
+    include Decidim::Loggable
+
     belongs_to :organization, foreign_key: "decidim_organization_id", class_name: "Decidim::Organization", inverse_of: :static_pages
 
     validates :slug, presence: true, uniqueness: { scope: :organization }
@@ -29,6 +32,10 @@ module Decidim
       DEFAULT_PAGES.include?(slug)
     end
 
+    def self.log_presenter_class_for(_log)
+      Decidim::AdminLog::StaticPagePresenter
+    end
+
     # Whether this is page is a default one or not.
     #
     # Returns Boolean.

data/app/models/decidim/user.rb

@@ -8,6 +8,7 @@ module Decidim
   class User < ApplicationRecord
     include Nicknamizable
     include Decidim::Followable
+    include Decidim::Loggable
 
     OMNIAUTH_PROVIDERS = [:facebook, :twitter, :google_oauth2, (:developer if Rails.env.development?)].compact
     ROLES = %w(admin user_manager).freeze
@@ -29,7 +30,6 @@ module Decidim
     validates :tos_agreement, acceptance: true, allow_nil: false, on: :create
     validates :avatar, file_size: { less_than_or_equal_to: ->(_record) { Decidim.maximum_avatar_size } }
     validates :email, :nickname, uniqueness: { scope: :organization }, unless: -> { deleted? || managed? || nickname.blank? }
-    validates :email, 'valid_email_2/email': { disposable: true }
 
     validate :all_roles_are_valid
 
@@ -44,6 +44,10 @@ module Decidim
     # Returns a String.
     attr_accessor :invitation_instructions
 
+    def self.log_presenter_class_for(_log)
+      Decidim::AdminLog::UserPresenter
+    end
+
     # Checks if the user has the given `role` or not.
     #
     # role - a String or a Symbol that represents the role that is being

data/app/models/decidim/user_group.rb

@@ -3,6 +3,9 @@
 module Decidim
   # A UserGroup is an organization of citizens
   class UserGroup < ApplicationRecord
+    include Decidim::Traceable
+    include Decidim::Loggable
+
     belongs_to :organization, foreign_key: "decidim_organization_id", class_name: "Decidim::Organization"
 
     has_many :memberships, class_name: "Decidim::UserGroupMembership", foreign_key: :decidim_user_group_id, dependent: :destroy
@@ -20,6 +23,10 @@ module Decidim
     scope :verified, -> { where.not(verified_at: nil) }
     scope :rejected, -> { where.not(rejected_at: nil) }
 
+    def self.log_presenter_class_for(_log)
+      Decidim::AdminLog::UserGroupPresenter
+    end
+
     # Public: Checks if the user group is verified.
     def verified?
       verified_at.present?

data/app/presenters/decidim/admin_log/feature_presenter.rb

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+module Decidim
+  module AdminLog
+    # This class holds the logic to present a `Decidim::Feature`
+    # for the `AdminLog` log.
+    #
+    # Usage should be automatic and you shouldn't need to call this class
+    # directly, but here's an example:
+    #
+    #    action_log = Decidim::ActionLog.last
+    #    view_helpers # => this comes from the views
+    #    FeaturePresenter.new(action_log, view_helpers).present
+    class FeaturePresenter < Decidim::Log::BasePresenter
+      private
+
+      def diff_fields_mapping
+        {
+          name: :i18n,
+          published_at: :date,
+          weight: :integer
+        }
+      end
+
+      def i18n_labels_scope
+        "activemodel.attributes.feature"
+      end
+
+      def action_string
+        case action
+        when "create", "delete", "publish", "unpublish"
+          "decidim.admin_log.feature.#{action}"
+        else
+          super
+        end
+      end
+
+      def has_diff?
+        action == "unpublish" || super
+      end
+    end
+  end
+end

data/app/presenters/decidim/admin_log/moderation_presenter.rb

@@ -0,0 +1,48 @@
+# frozen_string_literal: true
+
+module Decidim
+  module AdminLog
+    # This class holds the logic to present a `Decidim::Moderation`
+    # for the `AdminLog` log.
+    #
+    # Usage should be automatic and you shouldn't need to call this class
+    # directly, but here's an example:
+    #
+    #    action_log = Decidim::ActionLog.last
+    #    view_helpers # => this comes from the views
+    #    ModerationPresenter.new(action_log, view_helpers).present
+    class ModerationPresenter < Decidim::Log::BasePresenter
+      private
+
+      def diff_fields_mapping
+        {
+          hidden_at: :date,
+          report_count: :integer
+        }
+      end
+
+      def action_string
+        case action
+        when "hide", "unreport"
+          "decidim.admin_log.moderation.#{action}"
+        else
+          super
+        end
+      end
+
+      def i18n_labels_scope
+        "decidim.moderations.models.moderation.fields"
+      end
+
+      def i18n_params
+        super.merge(
+          resource_type: action_log.extra.dig("extra", "reportable_type").try(:demodulize)
+        )
+      end
+
+      def has_diff?
+        action == "unreport" || super
+      end
+    end
+  end
+end

data/app/presenters/decidim/admin_log/newsletter_presenter.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+module Decidim
+  module AdminLog
+    # This class holds the logic to present a `Decidim::Newsletter`
+    # for the `AdminLog` log.
+    #
+    # Usage should be automatic and you shouldn't need to call this class
+    # directly, but here's an example:
+    #
+    #    action_log = Decidim::ActionLog.last
+    #    view_helpers # => this comes from the views
+    #    NewsletterPresenter.new(action_log, view_helpers).present
+    class NewsletterPresenter < Decidim::Log::BasePresenter
+      private
+
+      def diff_fields_mapping
+        {
+          body: :i18n,
+          subject: :i18n
+        }
+      end
+
+      def action_string
+        case action
+        when "create", "delete", "deliver", "update"
+          "decidim.admin_log.newsletter.#{action}"
+        else
+          super
+        end
+      end
+
+      def i18n_labels_scope
+        "activemodel.attributes.newsletter"
+      end
+
+      # Private: Caches the object that will be responsible of presenting the newsletter.
+      # Overwrites the method so that we can use a custom presenter to show the correct
+      # path for the newsletter.
+      #
+      # Returns an object that responds to `present`.
+      def resource_presenter
+        @resource_presenter ||= Decidim::AdminLog::NewsletterResourcePresenter.new(action_log.resource, h, action_log.extra["resource"])
+      end
+    end
+  end
+end

data/app/presenters/decidim/admin_log/newsletter_resource_presenter.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module Decidim
+  module AdminLog
+    # This class extends the default resource presenter for logs, so that
+    # it can properly link to the newsletter.
+    class NewsletterResourcePresenter < Decidim::Log::ResourcePresenter
+      private
+
+      # Private: Finds the public link for the given static page..
+      #
+      # Returns an HTML-safe String.
+      def resource_path
+        @resource_path ||= h.decidim_admin.newsletter_path(resource)
+      end
+    end
+  end
+end

data/app/presenters/decidim/admin_log/organization_presenter.rb

@@ -0,0 +1,84 @@
+# frozen_string_literal: true
+
+module Decidim
+  module AdminLog
+    # This class holds the logic to present a `Decidim::Organization`
+    # for the `AdminLog` log.
+    #
+    # Usage should be automatic and you shouldn't need to call this class
+    # directly, but here's an example:
+    #
+    #    action_log = Decidim::ActionLog.last
+    #    view_helpers # => this comes from the views
+    #    OrganizationPresenter.new(action_log, view_helpers).present
+    class OrganizationPresenter < Decidim::Log::BasePresenter
+      private
+
+      def diff_fields_mapping
+        settings_attributes_mapping
+          .merge(omnipresent_banner_attributes_mapping)
+          .merge(highlighted_content_banner_attributes_mapping)
+          .merge(appearance_attributes_mapping)
+      end
+
+      def settings_attributes_mapping
+        {
+          name: :string,
+          default_locale: :locale,
+          reference_prefix: :string,
+          twitter_handler: :string,
+          facebook_handler: :string,
+          instagram_handler: :string,
+          youtube_handler: :string,
+          github_handler: :string
+        }
+      end
+
+      def omnipresent_banner_attributes_mapping
+        {
+          enable_omnipresent_banner: :boolean,
+          omnipresent_banner_url: :string,
+          omnipresent_banner_short_description: :i18n,
+          omnipresent_banner_title: :i18n
+        }
+      end
+
+      def highlighted_content_banner_attributes_mapping
+        {
+          highlighted_content_banner_enabled: :boolean,
+          highlighted_content_banner_action_url: :string,
+          highlighted_content_banner_image: :string,
+          highlighted_content_banner_title: :i18n,
+          highlighted_content_banner_short_description: :i18n,
+          highlighted_content_banner_action_title: :i18n,
+          highlighted_content_banner_action_subtitle: :i18n
+        }
+      end
+
+      def appearance_attributes_mapping
+        {
+          cta_button_path: :string,
+          cta_button_text: :i18n,
+          description: :i18n,
+          welcome_text: :i18n,
+          homepage_image: :string,
+          logo: :string,
+          header_snippets: :string,
+          favicon: :string,
+          official_img_header: :string,
+          official_img_footer: :string,
+          official_url: :string,
+          show_statistics: :boolean
+        }
+      end
+
+      def action_string
+        "decidim.admin_log.organization.update"
+      end
+
+      def i18n_labels_scope
+        "activemodel.attributes.organization"
+      end
+    end
+  end
+end

data/app/presenters/decidim/admin_log/scope_presenter.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+module Decidim
+  module AdminLog
+    # This class holds the logic to present a `Decidim::Scope`
+    # for the `AdminLog` log.
+    #
+    # Usage should be automatic and you shouldn't need to call this class
+    # directly, but here's an example:
+    #
+    #    action_log = Decidim::ActionLog.last
+    #    view_helpers # => this comes from the views
+    #    ScopePresenter.new(action_log, view_helpers).present
+    class ScopePresenter < Decidim::Log::BasePresenter
+      private
+
+      def diff_fields_mapping
+        {
+          code: :string,
+          name: :i18n,
+          parent_id: :scope,
+          scope_type_id: :scope_type
+        }
+      end
+
+      def action_string
+        case action
+        when "create", "delete", "update"
+          if parent_name.present?
+            "decidim.admin_log.scope.#{action}_with_parent"
+          else
+            "decidim.admin_log.scope.#{action}"
+          end
+        else
+          super
+        end
+      end
+
+      def i18n_labels_scope
+        "activemodel.attributes.scope"
+      end
+
+      def i18n_params
+        super.merge(
+          parent_scope: h.translated_attribute(parent_name)
+        )
+      end
+
+      def parent_name
+        action_log.extra.dig("extra", "parent_name")
+      end
+    end
+  end
+end