pettanr_unknown_v01_licenses 0.1.4 → 0.1.5

data/app/controllers/pettanr_unknown_v01_licenses/attributes_controller.rb

@@ -2,8 +2,7 @@ require_dependency "pettanr_unknown_v01_licenses/application_controller"
 
 module PettanrUnknownV01Licenses
   class AttributesController < ApplicationController
-    layout 'test' if MagicNumber['test_layout']
-    before_filter :authenticate_user!, :only => [:new, :create]
+    before_filter :authenticate_user, :only => [:new, :create]
     before_filter :authenticate_artist, :only => [:new, :create]
     
     def new
@@ -24,7 +23,7 @@ module PettanrUnknownV01Licenses
       @unknown_license = PettanrUnknownV01Licenses::Attribute.new params[:unknown_license]
       @license = License.show @unknown_license.license_id, @operators
       @resource_picture = @original_picture.resource_picture || ResourcePicture.new
-      @resource_picture.attributes = @unknown_license.resource_picture_attributes @original_picture
+      @resource_picture.attributes = @unknown_license.resource_picture_attributes @original_picture, @license
       @resource_picture.overwrite @original_picture
       respond_to do |format|
         if @unknown_license.valid?

data/app/models/pettanr_unknown_v01_licenses/attribute.rb

@@ -2,18 +2,26 @@ module PettanrUnknownV01Licenses
   class Attribute < ActiveRecord::Base
     belongs_to :license
     
-    validates :license_id, :presence => true, :numericality => true, :existence => true
+    validates :license_id, :presence => true, :numericality => true #, :existence => true
     validates :artist_name, :presence => true
     
     def supply_default operators
       self.artist_name = operators.artist.name
     end
     
-    def resource_picture_attributes op
+    def resource_picture_attributes op, ls
       {
-        :original_picture_id => op.id, :license_id => self.license.id, 
-        :artist_name => self.artist_name, :classname => self.license.license_group.classname, 
-        :credit => self.credit, :settings => self.license.settings
+        :original_picture_id => op.id, 
+        :license_id => ls.id, 
+        :artist_name => self.artist_name, 
+        :license_group_classname => ls.license_group_classname, 
+        :license_group_settings => ls.license_group_settings,
+        :credit_picture_settings => ls.credit_picture_settings, 
+        :license_settings => {
+          :system_picture_id => ls.system_picture_id,
+          :source_url => self.source_url, 
+          :note => self.note
+        }.to_json
       }
     end
     
@@ -24,13 +32,5 @@ module PettanrUnknownV01Licenses
       [op, oplg, lg]
     end
     
-    def credit
-      {
-        :system_picture_id => self.license.system_picture_id,
-        :source_url => self.source_url, 
-        :note => self.note
-      }.to_json.to_s
-    end
-    
   end
 end

data/app/views/pettanr_unknown_v01_licenses/attributes/_credit.html.erb

@@ -7,9 +7,10 @@
       %>
     </td>
     <td>
+      <%  picture.boosts 'post' %>
       <div class="md5">
         <%  picture.license.boosts 'read' %>
-        <% if picture.credit_data['source_url'].blank? %>
+        <% if picture.attribute_extend.source_url.blank? %>
           <% source_url_off_btn = SystemPicture.find(picture.license.credit_picture_extend.source_url_off_btn_id) %>
           <%= 
             tag(:img, :src => source_url_off_btn.url, :alt => 'no source', :width => source_url_off_btn.width, :height => source_url_off_btn.height)
@@ -18,12 +19,12 @@
           <% source_url_btn = SystemPicture.find(picture.license.credit_picture_extend.source_url_btn_id) %>
           <%= link_to(
             tag(:img, :src => source_url_btn.url, :alt => 'source url', :width => source_url_btn.width, :height => source_url_btn.height), 
-            picture.credit_data['source_url'] )
+            picture.attribute_extend.source_url)
           %>
         <% end %>
       </div>
       <div class="md5">
-        <%= h(truncate(picture.credit_data['note'], :length => 12)) %>
+        <%= h(truncate(picture.attribute_extend.note, :length => 12)) %>
       </div>
     </td>
   </tr>

data/app/views/pettanr_unknown_v01_licenses/attributes/_full_credit.html.erb

@@ -23,14 +23,14 @@
   <tr>
     <td><%= t_m 'PettanrUnknownV01Licenses::Attribute.source_url' -%></td>
     <td>
-      <% if picture.credit_data['source_url'].blank? %>
+      <% if picture.attribute_extend.source_url.blank? %>
       <% else %>
-        <%= link_to(h(picture.credit_data['source_url']), picture.credit_data['source_url']) -%></td>
+        <%= link_to(h(picture.attribute_extend.source_url), picture.attribute_extend.source_url) -%></td>
       <% end %>
     </td>
   </tr>
   <tr>
     <td><%= t_m 'PettanrUnknownV01Licenses::Attribute.note' -%></td>
-    <td><%= h picture.credit_data['note'] -%></td>
+    <td><%= h picture.attribute_extend.note -%></td>
   </tr>
 </table>

data/db/migrate/20140414094414_fix_pettanr_unknown_v01_licenses_attributes_system_picture_id.rb

@@ -0,0 +1,8 @@
+class FixPettanrUnknownV01LicensesAttributesSystemPictureId < ActiveRecord::Migration
+  def up
+    add_column :pettanr_unknown_v01_licenses_attributes, :system_picture_id, :integer, :null => false, :default => 0
+  end
+
+  def down
+  end
+end

data/lib/pettanr_unknown_v01_licenses.rb

@@ -53,21 +53,21 @@ module PettanrUnknownV01Licenses
       super
     end
     
-    def resource_picture_extend
-      self.new_resource_picture unless @resource_picture_extend
-      @resource_picture_extend ||= PettanrUnknownV01Licenses::Attribute.new
+    def attribute_extend
+      self.new_attribute unless @attribute_extend
+      @attribute_extend ||= PettanrUnknownV01Licenses::Attribute.new
     end
     
-    def resource_picture_extend= params
+    def attribute_extend= params
       # load attributes
-      @resource_picture_extend = PettanrUnknownV01Licenses::Attribute.new params
+      @attribute_extend = PettanrUnknownV01Licenses::Attribute.new params
       # no supply default params
     end
     
-    def new_resource_picture params = {} 
-      @resource_picture_extend = PettanrUnknownV01Licenses::Attribute.new params
+    def new_attribute params = {} 
+      @attribute_extend = PettanrUnknownV01Licenses::Attribute.new params
       # Attribute's supply_default need operators
-      # @resource_picture_extend.supply_default operators
+      # @attribute_extend.supply_default operators
     end
     
   end

data/lib/pettanr_unknown_v01_licenses/version.rb

@@ -1,3 +1,3 @@
 module PettanrUnknownV01Licenses
-  VERSION = "0.1.4"
+  VERSION = "0.1.5"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pettanr_unknown_v01_licenses
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-09 00:00:00.000000000 Z
+date: 2014-04-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -74,6 +74,7 @@ files:
 - db/migrate/20121007054509_create_pettanr_unknown_v01_licenses_attributes.rb
 - db/migrate/20140408085355_create_pettanr_unknown_v01_licenses_licenses.rb
 - db/migrate/20140408085413_create_pettanr_unknown_v01_licenses_credit_pictures.rb
+- db/migrate/20140414094414_fix_pettanr_unknown_v01_licenses_attributes_system_picture_id.rb
 - db/pettanr_unknown_v01_license.json
 - db/pettanr_unknown_v01_license.yml
 - db/Unknown.png