pettanr_public_domain_v01_licenses 0.1.5 → 0.1.6

data/app/controllers/pettanr_public_domain_v01_licenses/attributes_controller.rb

@@ -5,28 +5,37 @@ module PettanrPublicDomainV01Licenses
     before_filter :authenticate_user, :only => [:new, :create]
     before_filter :authenticate_artist, :only => [:new, :create]
     
+    def self.my_module
+      PettanrPublicDomainV01Licenses
+    end
+    
+    def self.my_model
+      PettanrPublicDomainV01Licenses::Attribute
+    end
+    
     def new
       @original_picture, @original_picture_license_group, @license_group = 
-        PettanrPublicDomainV01Licenses::Attribute.remember_params params, @operators
-      @public_domain_license = PettanrPublicDomainV01Licenses::Attribute.new
-      @public_domain_license.supply_default @operators
+        self.class.my_model.remember_params params, @operators
+      @attr = @license_group.my_engine.new_attribute {}
+      # rare case !  
+      # Attribute need artist name by @operators  
+      @attr.supply_default @operators
 
       respond_to do |format|
-        format.html # new.html.erb
-#        format.js
+        format.html
       end
     end
 
     def create
       @original_picture, @original_picture_license_group, @license_group = 
-        PettanrPublicDomainV01Licenses::Attribute.remember_params params, @operators
-      @public_domain_license = PettanrPublicDomainV01Licenses::Attribute.new params[:public_domain_license]
-      @license = License.show @public_domain_license.license_id, @operators
+        self.class.my_model.remember_params params, @operators
+      @attr = self.class.my_module.new_attribute params[:attributes]
+      @license = License.show @attr.license_id, @operators
       @resource_picture = @original_picture.resource_picture || ResourcePicture.new
-      @resource_picture.attributes = @public_domain_license.resource_picture_attributes @original_picture, @license
+      @resource_picture.attributes = @attr.resource_picture_attributes @original_picture, @license
       @resource_picture.overwrite @original_picture
       respond_to do |format|
-        if @public_domain_license.valid?
+        if @attr.valid?
           format.html { render main_app.new_resource_picture_path }
           format.json { render json: @resource_picture.to_json(ResourcePicture.show_json_opt), status: :created }
         else

data/app/models/pettanr_public_domain_v01_licenses/attribute.rb

@@ -1,9 +1,10 @@
 module PettanrPublicDomainV01Licenses
   class Attribute < ActiveRecord::Base
-    belongs_to :license
     
     validates :license_id, :presence => true, :numericality => true #, :existence => true
     validates :artist_name, :presence => true
+    # validates :source_url, :presence => true
+    # validates :note, :presence => true
     
     def supply_default operators
       self.artist_name = operators.artist.name
@@ -14,11 +15,14 @@ module PettanrPublicDomainV01Licenses
         :original_picture_id => op.id, 
         :license_id => ls.id, 
         :artist_name => self.artist_name, 
+        :system_picture_id => ls.system_picture_id,
         :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,
+          :license_id => ls.id,
+          # :system_picture_id => ls.system_picture_id, # no save
+          :artist_name => self.artist_name, 
           :source_url => self.source_url, 
           :note => self.note
         }.to_json
@@ -32,5 +36,25 @@ module PettanrPublicDomainV01Licenses
       [op, oplg, lg]
     end
     
+    def engine_name
+      "pettanr_public_domain_v01_licenses"
+    end
+    
+    def form_name
+      self.engine_name + '/attributes'
+    end
+    
+    def label_name field_name = nil
+      self.class.to_s + (field_name ? '.' + field_name : '')
+    end
+    
+    def trans_name field_name = nil
+      self.engine_name + '.' + field_name
+    end
+    
+    def tag_name field_name
+      'attributes[' + field_name + ']'
+    end
+    
   end
 end

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

@@ -1,9 +1,11 @@
 <table class="no-border">
   <tr>
     <td width="1px">
-      <%= link_to(
-        tag(:img, :src => picture.license.system_picture.url, :alt => h(picture.license.caption.to_s), :width => picture.license.system_picture.width, :height => picture.license.system_picture.height), 
-        picture.license.url )
+      <%= content_tag(:a, 
+        tag(:img, :src => picture.system_picture.url, 
+        :alt => h(picture.alt_name), 
+        :width => picture.system_picture.width, :height => picture.system_picture.height), 
+        :href => picture.license.url )
       %>
     </td>
     <td>
@@ -12,17 +14,19 @@
         <%= h(truncate(picture.artist_name, :length => 12)) %>
       </div>
       <div class="md5">
-        <%  picture.license.boosts 'read' %>
         <% if picture.attribute_extend.source_url.blank? %>
-          <% source_url_off_btn = SystemPicture.find(picture.license.credit_picture_extend.source_url_off_btn_id) %>
+          <% source_url_off_btn = picture.credit_picture_extend.source_url_off_btn %>
           <%= 
-            tag(:img, :src => source_url_off_btn.url, :alt => 'no source', :width => source_url_off_btn.width, :height => source_url_off_btn.height)
+            tag(:img, :src => source_url_off_btn.url, :alt => 'no source', 
+              :width => source_url_off_btn.width, :height => source_url_off_btn.height
+            )
           %>
         <% else %>
-          <% source_url_btn = SystemPicture.find(picture.license.credit_picture_extend.source_url_btn_id) %>
+          <% source_url_btn = picture.credit_picture_extend.source_url_btn %>
           <%= link_to(
-            tag(:img, :src => source_url_btn.url, :alt => 'source url', :width => source_url_btn.width, :height => source_url_btn.height), 
-            picture.attribute_extend.source_url)
+            tag(:img, :src => source_url_btn.url, :alt => 'source url', 
+              :width => source_url_btn.width, :height => source_url_btn.height
+            ), picture.attribute_extend.source_url)
           %>
         <% end %>
       </div>

data/app/views/pettanr_public_domain_v01_licenses/attributes/_form.html.erb

@@ -1,36 +1,30 @@
-<% 
-  unless @public_domain_license
-    @public_domain_license = PettanrPublicDomainV01Licenses::Attribute.new
-    @public_domain_license.supply_default @operators
-  end
- %>
-<%= form_tag('/pettanr_public_domain_v01_licenses/attributes') do %>
-  <%= render 'system/error_explanation', :obj => @public_domain_license %>
+<%= form_tag(@attr.form_name) do %>
+  <%= render 'system/error_explanation', :obj => @attr %>
 
   <table class="no-border">
     <tr>
       <td><%= t_m 'LicenseGroup' -%></td>
-      <td><%= h @license_group.caption -%></td>
-      <td><%= t('pettanr_public_domain_v01_licenses.selected') -%></td>
+      <td><%= t_m @attr.label_name -%></td>
+      <td><%= t(@attr.trans_name('selected')) -%></td>
     </tr>
     <tr>
-      <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute.license_id' -%></td>
-      <td><%= collection_select :public_domain_license, :license_id, @license_group.licenses.map {|l| [l.caption, l.id] }, :last, :first -%></td>
-      <td><%= t('pettanr_public_domain_v01_licenses.no_blank') -%></td>
+      <td><%= t_m @attr.label_name('license_id') -%></td>
+      <td><%= collection_select :attributes, :license_id, @license_group.licenses.map {|l| [l.caption, l.id] }, :last, :first -%></td>
+      <td><%= t(@attr.trans_name('no_blank')) -%></td>
     </tr>
     <tr>
-      <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute.artist_name' -%></td>
-      <td><%= text_field_tag 'public_domain_license[artist_name]', @public_domain_license.artist_name -%></td>
-      <td><%= t('pettanr_public_domain_v01_licenses.no_blank') -%></td>
+      <td><%= t_m @attr.label_name('artist_name') -%></td>
+      <td><%= text_field_tag @attr.tag_name('artist_name'), @attr.artist_name -%></td>
+      <td><%= t(@attr.trans_name('no_blank')) -%></td>
     </tr>
     <tr>
-      <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute.source_url' -%></td>
-      <td><%= text_field_tag 'public_domain_license[source_url]', @public_domain_license.source_url -%></td>
+      <td><%= t_m @attr.label_name('source_url') -%></td>
+      <td><%= text_field_tag @attr.tag_name('source_url'), @attr.source_url -%></td>
       <td></td>
     </tr>
     <tr>
-      <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute.note' -%></td>
-      <td><%= text_field_tag 'public_domain_license[note]', @public_domain_license.note -%></td>
+      <td><%= t_m @attr.label_name('note') -%></td>
+      <td><%= text_field_tag @attr.tag_name('note'), @attr.note -%></td>
       <td></td>
     </tr>
   </table>
@@ -40,6 +34,6 @@
   <%= hidden_field_tag 'original_picture_license_group[license_group_id]', @original_picture_license_group.license_group_id %>
 
   <div class="actions">
-    <%= submit_tag  t('pettanr_public_domain_v01_licenses.next') %>
+    <%= submit_tag t(@attr.trans_name('next')) %>
   </div>
 <% end %>

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

@@ -2,7 +2,9 @@
   <tr>
     <td colspan="2">
       <%= link_to(
-        tag(:img, :src => picture.license.system_picture.url, :alt => h(picture.license.license_group.caption.to_s + '[' + picture.license.caption.to_s + ']'), :width => picture.license.system_picture.width, :height => picture.license.system_picture.height), 
+        tag(:img, :src => picture.license.system_picture.url, 
+        :alt => h(picture.alt_name), 
+        :width => picture.system_picture.width, :height => picture.system_picture.height), 
         picture.license.url )
       %>
     </td>
@@ -13,19 +15,19 @@
     </td>
   </tr>
   <tr>
-    <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute' -%></td>
+    <td><%= t_m picture.attribute_extend.label_name() -%></td>
     <td><%= link_to(h(picture.license.license_group.caption), picture.license.license_group.url) -%></td>
   </tr>
   <tr>
-    <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute.license_id' -%></td>
+    <td><%= t_m picture.attribute_extend.label_name('license_id') -%></td>
     <td><%= link_to(h(picture.license.caption), picture.license.url) -%></td>
   </tr>
   <tr>
-    <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute.artist_name' -%></td>
+    <td><%= t_m picture.attribute_extend.label_name('artist_name') -%></td>
     <td><%= h(picture.artist_name) -%></td>
   </tr>
   <tr>
-    <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute.source_url' -%></td>
+    <td><%= t_m picture.attribute_extend.label_name('source_url') -%></td>
     <td>
       <% if picture.attribute_extend.source_url.blank? %>
       <% else %>
@@ -34,7 +36,7 @@
     </td>
   </tr>
   <tr>
-    <td><%= t_m 'PettanrPublicDomainV01Licenses::Attribute.note' -%></td>
+    <td><%= t_m picture.attribute_extend.label_name('note') -%></td>
     <td><%= h picture.attribute_extend.note -%></td>
   </tr>
 </table>

data/lib/pettanr_public_domain_v01_licenses.rb

@@ -54,21 +54,23 @@ module PettanrPublicDomainV01Licenses
     end
     
     def attribute_extend
-      self.new_attribute unless @attribute_extend
-      @attribute_extend ||= PettanrPublicDomainV01Licenses::Attribute.new
+      @attribute_extend
     end
     
     def attribute_extend= params
       # load attributes
-      @attribute_extend = PettanrPublicDomainV01Licenses::Attribute.new params
+      @attribute_extend = PettanrPublicDomainV01Licenses.new_attribute params
       # no supply default params
     end
     
-    def new_attribute params = {} 
-      @attribute_extend = PettanrPublicDomainV01Licenses::Attribute.new params
-      # Attribute's supply_default need operators
-      # @attribute_extend.supply_default operators
+  end
+  
+  module ModuleMethods
+    def new_attribute params = {}
+      PettanrPublicDomainV01Licenses::Attribute.new params
     end
     
   end
+
+  extend ModuleMethods
 end

data/lib/pettanr_public_domain_v01_licenses/version.rb

@@ -1,3 +1,3 @@
 module PettanrPublicDomainV01Licenses
-  VERSION = "0.1.5"
+  VERSION = "0.1.6"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pettanr_public_domain_v01_licenses
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-15 00:00:00.000000000 Z
+date: 2014-04-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -61,9 +61,7 @@ files:
 - app/models/pettanr_public_domain_v01_licenses/attribute.rb
 - app/models/pettanr_public_domain_v01_licenses/credit_picture.rb
 - app/models/pettanr_public_domain_v01_licenses/license.rb
-- app/views/layouts/pettanr_public_domain_v01_licenses/application.html.erb
 - app/views/pettanr_public_domain_v01_licenses/attributes/new.html.erb
-- app/views/pettanr_public_domain_v01_licenses/attributes/new.js.erb
 - app/views/pettanr_public_domain_v01_licenses/attributes/_confirm.html.erb
 - app/views/pettanr_public_domain_v01_licenses/attributes/_credit.html.erb
 - app/views/pettanr_public_domain_v01_licenses/attributes/_form.html.erb

data/app/views/layouts/pettanr_public_domain_v01_licenses/application.html.erb

@@ -1,14 +0,0 @@
-<!DOCTYPE html>
-<html>
-<head>
-  <title>PettanrPublicDomainV01Licenses</title>
-  <%= stylesheet_link_tag    "pettanr_public_domain_v01_licenses/application", :media => "all" %>
-  <%= javascript_include_tag "pettanr_public_domain_v01_licenses/application" %>
-  <%= csrf_meta_tags %>
-</head>
-<body>
-
-<%= yield %>
-
-</body>
-</html>

data/app/views/pettanr_public_domain_v01_licenses/attributes/new.js.erb

@@ -1 +0,0 @@
-$("#oplg").html("<%= escape_javascript(render('form')) -%>");