Merge pull request #49 from cortex-cms/refactor/COR-689/Switch-Asset-File-Naming-Structure

COR 689: Switch Asset File Naming Structure

Author: MKwenhua

app/models/asset_field_type.rb

@@ -36,6 +36,7 @@ def data
         'file_size': asset_file_size,
         'updated_at': asset_updated_at
       },
+      'media_title': media_title,
       'asset_field_type_id': id
     }
   end
@@ -74,6 +75,10 @@ def allowed_content_types
     end
   end
 
+  def media_title
+    existing_data['media_title'] || ContentItemService.form_fields[@metadata[:naming_data][:title]][:text].parameterize.underscore
+  end
+
   def mapping_field_name
     "#{field_name.parameterize('_')}_asset_file_name"
   end
@@ -128,11 +133,7 @@ def style_urls
 
   def existing_metadata
     metadata.except!(:existing_data)
-
-    unless existing_data.empty?
-      metadata[:path].gsub!(":id", existing_data['asset_field_type_id']) if metadata[:path]
-    end
-
+    metadata[:path].gsub!(":media_title", media_title) if metadata[:path]
     metadata
   end
 end

lib/tasks/cortex/core/media.rake

@@ -18,6 +18,8 @@ namespace :cortex do
         puts "Creating Fields..."
 
         allowed_asset_content_types = %w(txt css js pdf doc docx ppt pptx csv xls xlsx svg ico png jpg gif bmp)
+        fieldTitle = media.fields.new(name: 'Title', field_type: 'text_field_type', validations: {presence: true, uniqueness: true})
+        fieldTitle.save
         media.fields.new(name: 'Asset', field_type: 'asset_field_type',
                          validations:
                            {
@@ -29,6 +31,9 @@ namespace :cortex do
                            },
                          metadata:
                            {
+                             naming_data: {
+                               title: fieldTitle.id
+                             },
                              styles: {
                                large: {geometry: '1800x1800>', format: :jpg},
                                medium: {geometry: '800x800>', format: :jpg},
@@ -39,14 +44,14 @@ namespace :cortex do
                              },
                              processors: [:thumbnail, :paperclip_optimizer],
                              preserve_files: true,
-                             path: ':class/:attachment/careerbuilder-:style-:id.:extension',
+                             path: ':class/:attachment/:media_title-:style.:extension',
                              s3_headers: {'Cache-Control': 'public, max-age=315576000'}
                            })
-        media.fields.new(name: 'Title', field_type: 'text_field_type', validations: {presence: true})
         media.fields.new(name: 'Description', field_type: 'text_field_type', validations: {presence: true})
         media.fields.new(name: 'Tags', field_type: 'tag_field_type')
         media.fields.new(name: 'Expiration Date', field_type: 'date_time_field_type')
         media.fields.new(name: 'Alt Tag', field_type: 'text_field_type')
+
         media.save!
 
         puts "Creating Wizard Decorators..."
@@ -61,7 +66,7 @@ namespace :cortex do
                   "grid_width": 12,
                   "elements": [
                     {
-                      "id": media.fields[0].id
+                      "id": media.fields.find_by_name('Asset').id
                     }
                   ]
                 }
@@ -76,23 +81,23 @@ namespace :cortex do
                   "grid_width": 6,
                   "elements": [
                     {
-                      "id": media.fields[1].id
+                      "id": media.fields.find_by_name('Title').id
                     },
                     {
-                      "id": media.fields[2].id,
+                      "id": media.fields.find_by_name('Description').id,
                       "render_method": "multiline_input",
                       "display": {
                          "rows": 3
                       }
                     },
                     {
-                      "id": media.fields[3].id
+                      "id": media.fields.find_by_name('Tags').id
                     },
                     {
-                      "id": media.fields[4].id
+                      "id": media.fields.find_by_name('Expiration Date').id
                     },
                     {
-                      "id": media.fields[5].id
+                      "id": media.fields.find_by_name('Alt Tag').id
                     }
                   ]
                 },
@@ -163,7 +168,7 @@ namespace :cortex do
                 "cells": [
                   {
                     "field": {
-                      "id": media.fields[1].id
+                      "id": media.fields.find_by_name('Title').id
                     },
                     "display": {
                       "classes": [
@@ -174,7 +179,7 @@ namespace :cortex do
                   },
                   {
                     "field": {
-                      "id": media.fields[2].id
+                      "id": media.fields.find_by_name('Description').id
                     }
                   }
                 ]
@@ -184,7 +189,7 @@ namespace :cortex do
                 "cells": [
                   {
                     "field": {
-                      "id": media.fields[3].id
+                      "id": media.fields.find_by_name('Tags').id
                     },
                     "display": {
                       "classes": [