decidim-core 0.25.0 → 0.26.0.rc2

data/config/locales/sv.yml

@@ -76,6 +76,12 @@ sv:
       decidim_with_day_and_month_name: "%A %d %b %Y"
       decidim_with_month_name: "%d %B %Y"
       decidim_with_month_name_short: "%d %b"
+  datetime:
+    distance_in_words:
+      x_seconds:
+        one: 1 sek. sedan
+        other: "%{count} sek. sedan"
+        zero: just nu
   decidim:
     accessibility:
       external_link: Extern länk
@@ -98,6 +104,7 @@ sv:
         error: Det gick inte att radera ditt konto.
         success: Ditt konto har raderats.
       show:
+        available_locales_helper: Välj det språk du vill använda på plattformen och i de notiserna du får
         change_password: Ändra lösenord
         update_account: Uppdatera konto
       update:
@@ -255,6 +262,8 @@ sv:
         download: Hämta fil
       documents:
         related_documents: Relaterade dokument
+      geocoding:
+        geocoded_address: Adress
       photos:
         related_photos: Relaterade bilder
     author:
@@ -315,6 +324,8 @@ sv:
           authorize: Auktorisera med "%{authorization}"
           explanation: För att utföra denna åtgärd måste du vara auktoriserad med "%{authorization}".
           title: Auktorisering krävs
+        ok:
+          title: Du har blivit auktoriserad på den här sidan. Ladda om sidan för att utföra din åtgärd
         pending:
           explanation: För att kunna utföra denna åtgärd måste du vara auktoriserad med "%{authorization}", men din auktorisering behandlas fortfarande
           resume: Se hur auktoriseringen med "%{authorization}" pågår
@@ -500,6 +511,11 @@ sv:
           this_application_will_not_be_able_to: 'Det här programmet kommer inte att kunna:'
           update_profile: Uppdatera din profil
           wants_to_use_your_account_html: "<strong>%{application_name}</strong> vill använda ditt konto"
+    editor_images:
+      create:
+        error: Fel vid uppladdning av bild
+        success: Bilden har laddats upp
+      drag_and_drop_help: Lägg till bilder genom att dra och släppa eller klistra in dem.
     endorsable:
       endorsements: Instämmanden
       endorsements_count: Antal Endorsements
@@ -645,6 +661,10 @@ sv:
           email_intro: En administratör tog bort din %{resource_type} eftersom den har rapporterats som %{report_reasons}.
           email_outro: Du har fått det här meddelandet eftersom du är en författare till detta innehåll.
           email_subject: Din %{resource_type} har tagits bort
+          notification_title: |-
+            En administratör tog bort din %{resource_type} eftersom den har rapporterats som %{report_reasons}.
+            </br>
+            <i>%{resource_content}</i>
       resource_endorsed:
         email_intro: '%{endorser_name} %{endorser_nickname}, som du följer, har just instämt med "%{resource_title}" och vi tror att du kan vara intresserad. Ta en titt och bidra:'
         email_outro: Du har fått det här meddelandet eftersom du följer %{endorser_nickname}. Du kan sluta att ta emot meddelanden via föregående länk.
@@ -658,6 +678,7 @@ sv:
           notification_title: <a href="%{resource_path}">Profilsidan</a> för %{name} (%{nickname}), som du följer, har uppdaterats.
     export_mailer:
       data_portability_export:
+        click_button: 'Hämta dina data genom att klicka på Nästa.<br/>Filen kommer vara tillgänglig till och med %{date}.<br/>Du kan använda <a href="https://www.7-zip.org/">7-Zip</a> (för Windows), <a href="https://www.keka.io/en/">Keka</a> (för MacOS) eller <a href="https://peazip.github.io">PeaZip</a> (för Linux) för att öppna den. Lösenord: %{password}'
         download: Hämta
       export:
         ready: En komprimerad version av exporten är bifogad.
@@ -686,10 +707,12 @@ sv:
       no_followers: Inga följare än.
     following:
       no_followings: Följer inte någon eller något än.
+      non_public_followings: Vissa resurser som följs är inte offentliga.
     follows:
       create:
         button: Följ
         error: Det gick inte att följa resursen.
+        participatory_space: Följer <span class="show-for-sr">%{resource_name}</span>
       destroy:
         button: Sluta följa
         error: Det gick inte att sluta följa den här resursen.
@@ -706,6 +729,7 @@ sv:
         file:
           explanation: 'Vägledning för fil:'
           message_1: Måste vara en bild eller ett dokument.
+          message_2: Använd helst landskapsbilder. Tjänsten beskär bilden. För CSV filer måste separatorn mellan kolumner vara ett kommatecken (",")
         image:
           explanation: 'Vägledning för bild:'
           message_1: Helst en landskapsbild som inte har någon text.
@@ -848,7 +872,12 @@ sv:
     links:
       invalid_url: Ogiltig URL
       warning:
+        body_1: Du är på väg att besöka en extern länk och vi vill att du ska vara försiktig med innehållet på den externa webbplatsen.
+        body_2: Kontrollera länken du är på väg att besöka och se till att den är säker innan du fortsätter.
+        cancel: Avbryt
         close_modal: Stäng fönster
+        proceed: Fortsätt
+        title: Öppna extern länk
     log:
       base_presenter:
         create: "%{user_name} skapade %{resource_name}"
@@ -931,6 +960,8 @@ sv:
           error: Konversationen har inte startat. Försök igen senare
         error_modal:
           close: Stäng fönster
+          correct_errors: Rätta alla fel och försök igen.
+          intro: 'De följande fel fanns i ditt meddelande:'
           ok: OK
         index:
           ago: sedan
@@ -942,6 +973,7 @@ sv:
           next: Nästa
           no_conversations: Du har inga konversationer än
           title: Samtal
+          to: Till
         reply:
           placeholder: Ditt svar...
           send: Skicka
@@ -949,10 +981,13 @@ sv:
         show:
           back: Tillbaka till alla konversationer
           chat_with: Konversation med
+          not_allowed: Deltagaren tar inte emot direktmeddelanden.
           title: Samtal med %{usernames}
         start:
           send: Skicka
           title: Starta en konversation
+        update:
+          error: Meddelandet skickades inte på grund av ett fel
     metrics:
       download:
         csv: Ladda ner data (CSV)
@@ -1020,12 +1055,13 @@ sv:
         administrators: Administratörer
         allow_public_contact: Tillåt alla att skicka direktmeddelanden till mig, även de som jag inte följer.
         direct_messages: Ta emot direktmeddelanden från alla
-        email_on_notification: Jag vill få ett e-brev varje gång jag får ett meddelande.
+        email_on_moderations: Jag vill få ett e-post varje gång något anmäls för moderering.
+        email_on_notification: Jag vill få ett e-post varje gång jag får en notis.
         everything_followed: Allt jag följer
         newsletter_notifications: Jag vill få nyhetsbrev
         newsletters: Nyhetsbrev
         own_activity: Min egen verksamhet, som när någon kommenterar mitt förslag eller nämner mig
-        receive_notifications_about: Jag vill få meddelanden om
+        receive_notifications_about: Jag vill få notiser om
         send_notifications_by_email: Skicka meddelanden via e-post
         update_notifications_settings: Spara ändringar
       update:
@@ -1098,6 +1134,7 @@ sv:
       not_allowed: Du har inte rätt att se det här innehållet
     profile:
       deleted: Deltagaren har raderats
+      inaccessible_message: Denna profil kan inte ses på grund av överträdelse av användarvillkoren!
       view: Visa
     profiles:
       default_officialization_text_for_user_groups: Den här gruppen är offentligt verifierad, namnet har kontrollerats att överensstämma med dess riktiga namn
@@ -1224,6 +1261,9 @@ sv:
       flag_user_modal:
         already_reported: Det här innehållet har redan rapporterats och kommer att granskas av en administratör.
         close: Stäng
+        description: Vad är olämpligt med användaren?
+        does_not_belong: Innehåller olaglig aktivitet, självmordshot, personlig information eller något annat du tycker inte hör hemma på %{organization_name}.
+        offensive: Innehåller rasism, sexism, skällsord, personliga attacker, dödshot, uppmaningar att begå självmord eller någon form av hatpropaganda.
         report: Rapportera
         spam: Innehåller klickbete, reklam, bedrägerier eller skriptbottar.
         title: Rapportera olämplig användare
@@ -1240,6 +1280,8 @@ sv:
         filters:
           areas: Områden
           select_an_area: Välj ett område
+      public_participation:
+        public_participation: Gör mitt deltagande synligt
       reference:
         reference: 'Referens: %{reference}'
       represent_user_group:
@@ -1322,6 +1364,7 @@ sv:
         body_2: 'Anledning: %{reason}'
         greetings: Hälsningar,<br/>%{organization_name}<br/><a href="%{organization_url}">%{organization_url}</a>
         hello: Hej %{admin},
+        subject: En ny användare har rapporterats på %{organization_name}
     version:
       show:
         back_to_resource: Gå tillbaka

data/config/locales/val-ES.yml

@@ -0,0 +1 @@
+val:

data/config/routes.rb

@@ -156,6 +156,8 @@ Decidim::Core::Engine.routes.draw do
     end
   end
 
+  resources :editor_images, only: [:create]
+
   namespace :gamification do
     resources :badges, only: [:index]
   end

data/db/migrate/20210730112319_create_decidim_editor_images.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+class CreateDecidimEditorImages < ActiveRecord::Migration[6.0]
+  def change
+    create_table :decidim_editor_images do |t|
+      t.references :decidim_author, null: false, foreign_key: { to_table: :decidim_users }, index: { name: "decidim_editor_images_author" }
+      t.references :decidim_organization, null: false, foreign_key: true, index: { name: "decidim_editor_images_constraint_organization" }
+
+      t.timestamps
+    end
+  end
+end

data/db/migrate/20211126183540_add_timestamps_to_content_blocks.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+class AddTimestampsToContentBlocks < ActiveRecord::Migration[6.0]
+  def up
+    add_timestamps :decidim_content_blocks, default: Time.zone.now
+    change_column_default :decidim_content_blocks, :created_at, nil
+    change_column_default :decidim_content_blocks, :updated_at, nil
+  end
+
+  def down
+    remove_column :decidim_content_blocks, :updated_at
+    remove_column :decidim_content_blocks, :created_at
+  end
+end

data/db/seeds.rb

@@ -132,21 +132,23 @@ if !Rails.env.production? || ENV["SEED"]
     admin_terms_accepted_at: Time.current
   )
 
-  regular_user = Decidim::User.find_or_initialize_by(email: "user@example.org")
+  ["user@example.org", "user2@example.org"].each do |email|
+    Decidim::User.find_or_initialize_by(email: email).update!(
+      name: Faker::Name.name,
+      nickname: Faker::Twitter.unique.screen_name,
+      password: "decidim123456",
+      password_confirmation: "decidim123456",
+      confirmed_at: Time.current,
+      locale: I18n.default_locale,
+      organization: organization,
+      tos_agreement: true,
+      personal_url: Faker::Internet.url,
+      about: Faker::Lorem.paragraph(sentence_count: 2),
+      accepted_tos_version: organization.tos_version
+    )
+  end
 
-  regular_user.update!(
-    name: Faker::Name.name,
-    nickname: Faker::Twitter.unique.screen_name,
-    password: "decidim123456",
-    password_confirmation: "decidim123456",
-    confirmed_at: Time.current,
-    locale: I18n.default_locale,
-    organization: organization,
-    tos_agreement: true,
-    personal_url: Faker::Internet.url,
-    about: Faker::Lorem.paragraph(sentence_count: 2),
-    accepted_tos_version: organization.tos_version
-  )
+  regular_user = Decidim::User.find_or_initialize_by(email: "user@example.org")
 
   locked_user = Decidim::User.find_or_initialize_by(email: "locked_user@example.org")
 

data/lib/decidim/api/functions/user_entity_finder.rb

@@ -22,7 +22,8 @@ module Decidim
           filters[argument.to_sym] = v
         end
         Decidim::UserBaseEntity
-          .where.not(confirmed_at: nil)
+          .confirmed
+          .not_blocked
           .find_by(filters)
       end
     end

data/lib/decidim/api/functions/user_entity_list.rb

@@ -19,7 +19,9 @@ module Decidim
       def call(_obj, args, ctx)
         @query = Decidim::UserBaseEntity
                  .where(organization: ctx[:current_organization])
-                 .where.not(confirmed_at: nil)
+                 .confirmed
+                 .not_blocked
+                 .includes(avatar_attachment: :blob)
         add_filter_keys(args[:filter])
         add_order_keys(args[:order].to_h)
         @query

data/lib/decidim/api/input_sorts/component_input_sort.rb

@@ -23,7 +23,7 @@ module Decidim
                prepare: lambda { |direction, ctx|
                           lambda { |locale|
                             locale = ctx[:current_organization].default_locale if locale.blank?
-                            [Arel.sql("name->? #{direction.upcase}"), locale]
+                            [Arel.sql("name->? #{direction.upcase}").to_s, locale]
                           }
                         }
     end

data/lib/decidim/common_passwords.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+module Decidim
+  class CommonPasswords
+    include Singleton
+
+    attr_reader :passwords
+
+    URLS = %w(
+      https://raw.githubusercontent.com/danielmiessler/SecLists/master/Passwords/xato-net-10-million-passwords-1000000.txt
+      https://raw.githubusercontent.com/danielmiessler/SecLists/master/Passwords/darkweb2017-top10000.txt
+      https://raw.githubusercontent.com/danielmiessler/SecLists/master/Passwords/Common-Credentials/10-million-password-list-top-1000000.txt
+    ).freeze
+
+    def initialize
+      raise FileNotFoundError unless File.exist?(self.class.common_passwords_path)
+
+      File.open(self.class.common_passwords_path, "r") do |file|
+        @passwords = file.read.split
+      end
+    end
+
+    def self.update_passwords!
+      File.open(common_passwords_path, "w") do |file|
+        common_password_list.each { |item| file.puts(item) }
+      end
+    end
+
+    def self.common_password_list
+      @common_password_list ||= begin
+        list = []
+        URLS.each do |url|
+          URI.open(url) do |data|
+            data.read.split.each do |line|
+              list << line if line.length >= min_length
+            end
+          end
+        end
+
+        list.uniq
+      end
+    end
+
+    def self.min_length
+      return ::PasswordValidator::MINIMUM_LENGTH if defined?(::PasswordValidator)
+
+      10
+    end
+
+    def self.common_passwords_path
+      File.join(__dir__, "db", "common-passwords.txt")
+    end
+
+    class FileNotFoundError < StandardError; end
+  end
+end

data/lib/decidim/content_parsers/inline_images_parser.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+module Decidim
+  module ContentParsers
+    # A parser that searches for inline images in an html content and
+    # replaces them with EditorImage attachments. Note that rewrite method may
+    # create EditorImage instances
+    #
+    # @see BaseParser Examples of how to use a content parser
+    class InlineImagesParser < BaseParser
+      # @return [String] the content with the inline images replaced.
+      def rewrite
+        return content unless inline_images?
+
+        replace_inline_images
+        parsed_content.to_html
+      end
+
+      def inline_images?
+        parsed_content.search(:img).find do |image|
+          image.attr(:src).start_with?(%r{data:image/[a-z]{3,4};base64,})
+        end
+      end
+
+      private
+
+      def parsed_content
+        @parsed_content ||= Nokogiri::HTML(content)
+      end
+
+      def replace_inline_images
+        parsed_content.search(:img).each do |image|
+          next unless image.attr(:src).start_with?(%r{data:image/[a-z]{3,4};base64,})
+
+          file = base64_tempfile(image.attr(:src))
+          editor_image = EditorImage.create!(
+            decidim_author_id: context[:user]&.id,
+            organization: context[:user].organization,
+            file: file
+          )
+
+          image.set_attribute(:src, editor_image.attached_uploader(:file).path)
+        end
+      end
+
+      def base64_tempfile(base64_data, filename = nil)
+        return base64_data unless base64_data.is_a? String
+
+        start_regex = %r{data:image/[a-z]{3,4};base64,}
+        filename ||= SecureRandom.hex
+
+        regex_result = start_regex.match(base64_data)
+
+        return unless base64_data && regex_result
+
+        start = regex_result.to_s
+        tempfile = Tempfile.new(filename)
+        tempfile.binmode
+        tempfile.write(Base64.decode64(base64_data[start.length..-1]))
+        ActionDispatch::Http::UploadedFile.new(
+          tempfile: tempfile,
+          filename: filename,
+          original_filename: filename
+        )
+      end
+    end
+  end
+end

data/lib/decidim/content_parsers.rb

@@ -8,5 +8,6 @@ module Decidim
     autoload :HashtagParser, "decidim/content_parsers/hashtag_parser"
     autoload :NewlineParser, "decidim/content_parsers/newline_parser"
     autoload :LinkParser, "decidim/content_parsers/link_parser"
+    autoload :InlineImagesParser, "decidim/content_parsers/inline_images_parser"
   end
 end

data/lib/decidim/content_renderers/link_renderer.rb

@@ -2,6 +2,7 @@
 
 module Decidim
   module ContentRenderers
+    # Original: https://github.com/neighborland/anchored/
     # A renderer that converts URLs to links and strips attributes in anchors.
     #
     # Examples:
@@ -20,7 +21,90 @@ module Decidim
         return content unless content.is_a?(String)
 
         options = { target: "_blank", rel: "nofollow noopener" }.merge(options)
-        Anchored::Linker.auto_link(content, options)
+        auto_link(content, options)
+      end
+
+      def auto_link(text, options = {}, &block)
+        return "" if text.to_s.empty?
+
+        auto_link_urls(text, options, &block)
+      end
+
+      # remove_target_if_local("http://same.com/x", "same.com", target: "_blank")
+      # => <a href="http://same.com/x">http://same.com/x</a>
+      #
+      # remove_target_if_local("http://same.com/x", "different.com", target: "_blank")
+      # => <a href="http://same.com/x" target="_blank">http://same.com/x</a>
+      #
+      # modifies options in place
+      def remove_target_if_local(href, domain, options)
+        return unless options[:target]
+
+        options.delete(:target) if href.include?("//#{domain}")
+      end
+
+      private
+
+      AUTO_LINK_RE = %r{(?: ((?:ftp|http|https):)// | www\. )[^\s<\u00A0"]+}ix.freeze
+
+      # # regexps for determining context, used high-volume
+      AUTO_LINK_CRE = [/<[^>]+$/, /^[^>]*>/, /<a\b.*?>/i, %r{</a>}i].freeze
+
+      PUNCTUATION_RE = %r{[^\p{Word}/=&]$}.freeze
+
+      BRACKETS = { "]" => "[", ")" => "(", "}" => "{" }.freeze
+
+      # Turns all urls into clickable links.  If a block is given, each url
+      # is yielded and the result is used as the link text.
+      def auto_link_urls(text, options = {})
+        # to_str is for SafeBuffer objects (text marked html_safe)
+        text.to_str.gsub(AUTO_LINK_RE) do
+          match = Regexp.last_match
+          href = match[0]
+          scheme = match[1]
+          punctuation = []
+
+          if auto_linked?(match)
+            # do not change string; URL is already linked
+            href
+          else
+            # don't include trailing punctuation character as part of the URL
+            while href.sub!(PUNCTUATION_RE, "")
+              punctuation.push Regexp.last_match(0)
+              if (opening = BRACKETS[punctuation.last]) && href.scan(opening).size > href.scan(punctuation.last).size
+                href << punctuation.pop
+                break
+              end
+            end
+
+            link_text = block_given? ? yield(href) : href
+            href = "http://#{href}" unless scheme
+
+            # content_tag(:a, link_text, html.merge(href: href)) + punctuation.reverse.join('')
+            anchor_tag(href, link_text, options) + punctuation.reverse.join
+          end
+        end
+      end
+
+      # Detects already linked context or position in the middle of a tag
+      # Note: this changes the current Regexp
+      def auto_linked?(match)
+        left = match.pre_match
+        right = match.post_match
+        (left =~ AUTO_LINK_CRE[0] && right =~ AUTO_LINK_CRE[1]) ||
+          (left.rindex(AUTO_LINK_CRE[2]) && Regexp.last_match.post_match !~ AUTO_LINK_CRE[3])
+      end
+
+      def anchor_attrs(options)
+        options.map { |k, v| %(#{k}="#{v}") }.unshift("").join(" ")
+      end
+
+      def anchor_tag(href, text, options)
+        options = options.dup
+        if (domain = options.delete(:domain))
+          remove_target_if_local href, domain, options
+        end
+        %(<a href="#{href}"#{anchor_attrs(options)}>#{text}</a>)
       end
     end
   end

data/lib/decidim/content_renderers/user_group_renderer.rb

@@ -10,7 +10,7 @@ module Decidim
     # @see BaseRenderer Examples of how to use a content renderer
     class UserGroupRenderer < BaseRenderer
       # Matches a global id representing a Decidim::UserGroup
-      GLOBAL_ID_REGEX = %r{gid://\S+/Decidim::UserGroup/\d+}.freeze
+      GLOBAL_ID_REGEX = %r{gid://[\w-]+/Decidim::UserGroup/\d+}.freeze
 
       # Replaces found Global IDs matching an existing user with
       # a link to their profile. The Global IDs representing an

data/lib/decidim/content_renderers/user_renderer.rb

@@ -10,7 +10,7 @@ module Decidim
     # @see BaseRenderer Examples of how to use a content renderer
     class UserRenderer < BaseRenderer
       # Matches a global id representing a Decidim::User
-      GLOBAL_ID_REGEX = %r{gid://\S+/Decidim::User/\d+}.freeze
+      GLOBAL_ID_REGEX = %r{gid://[\w-]+/Decidim::User/\d+}.freeze
 
       # Replaces found Global IDs matching an existing user with
       # a link to their profile. The Global IDs representing an

data/lib/decidim/core/engine.rb

@@ -17,7 +17,6 @@ require "rectify"
 require "carrierwave"
 require "rails-i18n"
 require "date_validator"
-require "truncato"
 require "file_validators"
 require "omniauth"
 require "omniauth-facebook"
@@ -34,11 +33,9 @@ require "cell/partial"
 require "kaminari"
 require "doorkeeper"
 require "doorkeeper-i18n"
-require "nobspw"
 require "batch-loader"
-require "etherpad-lite"
+require "mime-types"
 require "diffy"
-require "anchored"
 require "social-share-button"
 require "ransack"
 require "searchlight"
@@ -85,8 +82,6 @@ module Decidim
       end
 
       initializer "decidim.graphql_api" do
-        # Enable them method `!` everywhere for compatibility, this line will be removed when upgrading to GraphQL 2.0
-        GraphQL::DeprecatedDSL.activate
         Decidim::Api::QueryType.include Decidim::QueryExtensions
 
         Decidim::Api.add_orphan_type Decidim::Core::UserType
@@ -304,7 +299,7 @@ module Decidim
 
       initializer "SSL and HSTS" do
         Rails.application.configure do
-          config.force_ssl = Rails.env.production? && Decidim.config.force_ssl
+          config.force_ssl = Decidim.config.force_ssl
         end
       end
 
@@ -315,7 +310,7 @@ module Decidim
       end
 
       initializer "Expire sessions" do
-        Rails.application.config.session_store :cookie_store, expire_after: Decidim.config.expire_session_after
+        Rails.application.config.session_store :cookie_store, secure: Decidim.config.force_ssl, expire_after: Decidim.config.expire_session_after
       end
 
       initializer "decidim.core.register_resources" do
@@ -542,14 +537,14 @@ module Decidim
         end
       end
 
-      initializer "nbspw" do
-        NOBSPW.configuration.use_ruby_grep = true
-      end
-
       initializer "decidim.premailer" do
         Premailer::Adapter.use = :decidim
       end
 
+      initializer "decidim_core.webpacker.assets_path" do
+        Decidim.register_assets_path File.expand_path("app/packs", root)
+      end
+
       config.to_prepare do
         FoundationRailsHelper::FlashHelper.include Decidim::FlashHelperExtensions
       end

data/lib/decidim/core/test/factories.rb

@@ -248,7 +248,7 @@ FactoryBot.define do
   end
 
   factory :user_group_membership, class: "Decidim::UserGroupMembership" do
-    user
+    user { create(:user, :confirmed, organization: user_group.organization) }
     role { :creator }
     user_group
   end
@@ -749,4 +749,10 @@ FactoryBot.define do
       last_used_at { 1.hour.ago }
     end
   end
+
+  factory :editor_image, class: "Decidim::EditorImage" do
+    organization
+    author { create(:user, :admin, :confirmed, organization: organization) }
+    file { Decidim::Dev.test_file("city.jpeg", "image/jpeg") }
+  end
 end