Merge branch 'master' into 'master'

Master

add update modules feature and fix error

See merge request !3

app/controllers/personal_conferences_controller.rb

@@ -1,60 +1,54 @@
 class PersonalConferencesController < ApplicationController
-  def search_all_words(target,word)
-    target=target.upcase
-    words=word.upcase.split(' ')
-    return words.select{|value| target.include? value}==words
-  end
   def index
     params = OrbitHelper.params
-    writing_conferences = WritingConference.where(:is_hidden=>false).sort_for_frontend.page(OrbitHelper.params[:page_no]).per(OrbitHelper.page_data_count)
-    page = Page.where(:page_id => params[:page_id]).first rescue nil
-    if page.custom_string_field == "table"
+    page_data_count = OrbitHelper.page_data_count
+    writing_conferences = WritingConference.where(is_hidden: false).sort_for_frontend.page(OrbitHelper.params[:page_no]).per(page_data_count)
+    page = Page.where(page_id: params[:page_id]).first rescue nil
+    if page.custom_string_field == 'table'
       fields_to_show = page.custom_array_field rescue []
-      fields_to_show = [
-        "authors",
-        "paper_title",
-        "conference_title",
-        "location",
-        "paper_types",
-        "paper_levels"
-      ] if fields_to_show.blank?
-    else
-      fields_to_show = [
-        "year",
-        "paper_title"
+      if fields_to_show.blank?
+        fields_to_show = %w[
+          authors
+          paper_title
+          conference_title
+          location
+          paper_types
+          paper_levels
         ]
       end
-    if params[:selectbox] !=nil
-      writing_conferences_temp = WritingConference.where(:is_hidden=>false).sort_by{ |tp| [-tp[:year].to_i,-tp[:publication_date].to_i] }
+    else
+      fields_to_show = %w[
+        year
+        paper_title
+      ]
+    end
+    if !params[:selectbox].nil?
+      writing_conferences_temp = WritingConference.where(is_hidden: false).sort_by { |tp| [-tp[:year].to_i, -tp[:publication_date].to_i] }
       case params[:selectbox]
-      when "paper_title","default"
-        if  page.custom_string_field == "table"
+      when 'paper_title', 'default'
+        if  page.custom_string_field == 'table'
           writing_conferences_show = writing_conferences_temp.select { |value| search_all_words(value.paper_title, params[:keywords]) }
         else
           writing_conferences_show = writing_conferences_temp.select { |value| search_all_words(value.create_link, params[:keywords]) }
         end
-      when "paper_levels"
-          writing_conferences_show = writing_conferences_temp.select {|value| search_all_words(value.conference_paper_levels.collect{|x| x.title}.join(', ').to_s, params[:keywords])}
-      when "paper_types"
-          writing_conferences_show = writing_conferences_temp.select {|value| search_all_words(value.conference_paper_types.collect{|x| x.title}.join(', ').to_s, params[:keywords])}
-      when "period"
+      when 'paper_levels'
+        writing_conferences_show = writing_conferences_temp.select { |value| search_all_words(value.conference_paper_levels.collect(&:title).join(', ').to_s, params[:keywords]) }
+      when 'paper_types'
+        writing_conferences_show = writing_conferences_temp.select { |value| search_all_words(value.conference_paper_types.collect(&:title).join(', ').to_s, params[:keywords]) }
+      when 'period'
         writing_conferences_show = writing_conferences_temp.select { |value| search_all_words(value.duration, params[:keywords]) }
-      when "publication_date"
-          writing_conferences_show = writing_conferences_temp.select {|value| search_all_words((value.send("publication_date").strftime("%Y/%m/%d") rescue ""), params[:keywords])}
-      when "author_type"
-          writing_conferences_show = writing_conferences_temp.select {|value| search_all_words(value.conference_author_types.collect{|cat| cat.title}.join(", "), params[:keywords])}
-      when "language"
-          writing_conferences_show = writing_conferences_temp.select {|value| search_all_words((!value.language.nil? ? t("#{value.language}") : ""), params[:keywords])}
+      when 'publication_date'
+        writing_conferences_show = writing_conferences_temp.select { |value| search_all_words((value.send('publication_date').strftime('%Y/%m/%d') rescue ''), params[:keywords]) }
+      when 'author_type'
+        writing_conferences_show = writing_conferences_temp.select { |value| search_all_words(value.conference_author_types.collect(&:title).join(', '), params[:keywords]) }
+      when 'language'
+        writing_conferences_show = writing_conferences_temp.select { |value| search_all_words((!value.language.nil? ? t(value.language.to_s) : ''), params[:keywords]) }
       else
         writing_conferences_show = writing_conferences_temp.select { |value| search_all_words(value.send(params[:selectbox]).to_s, params[:keywords]) }
       end
-      if params[:page_no].nil?
-          page_to_show = 1
-      else
-          page_to_show = params[:page_no].to_i
-      end
-      writing_conferences = writing_conferences_show[(page_to_show-1)*OrbitHelper.page_data_count...page_to_show*OrbitHelper.page_data_count]
-      writing_conferences_total_pages = (writing_conferences_show.length/OrbitHelper.page_data_count.to_f).ceil
+      page_to_show = params[:page_no].nil? ? 1 : params[:page_no].to_i
+      writing_conferences = writing_conferences_show[(page_to_show - 1) * page_data_count...page_to_show * page_data_count]
+      writing_conferences_total_pages = (writing_conferences_show.length / page_data_count.to_f).ceil
     else
       writing_conferences_total_pages = writing_conferences.total_pages
     end
@@ -63,135 +57,142 @@ class PersonalConferencesController < ApplicationController
       t = []
       fields_to_show.each do |fs|
         case fs
-        when "paper_title"
-          if  page.custom_string_field == "table"
-           t << {"value" => "<a href='#{OrbitHelper.url_to_show(writing_conference.to_param)}'>#{writing_conference.paper_title}</a>"}
+        when 'paper_title'
+          if  page.custom_string_field == 'table'
+            t << { 'value' => "<a href='#{OrbitHelper.url_to_show(writing_conference.to_param)}'>#{writing_conference.paper_title}</a>" }
           else
-           t << {"value" => "<a href='#{OrbitHelper.url_to_show(writing_conference.to_param)}'>#{writing_conference.create_link}</a>"}
+            t << { 'value' => "<a href='#{OrbitHelper.url_to_show(writing_conference.to_param)}'>#{writing_conference.create_link}</a>" }
           end
-        when "paper_levels"
-          t << {"value" => ( !writing_conference.conference_paper_levels.blank? ? "(#{writing_conference.conference_paper_levels.collect{|x| x.title}.join(', ')})" : nil)}
-        when "paper_types"
-          t << {"value" => ( !writing_conference.conference_paper_types.blank? ? "(#{writing_conference.conference_paper_types.collect{|x| x.title}.join(', ')})" : nil)} 
-        when "publication_date"
-          t << {"value" => (writing_conference.send(fs).strftime("%Y/%m") rescue "")}
-        when "author_type"
-          t << {"value" => (writing_conference.conference_author_types.collect{|cat| cat.title}.join(", ") rescue "")}
-        when "period"
-          t << {"value" => writing_conference.duration}
-        when "language"
-          t << {"value" => (!writing_conference.language.nil? ? t("#{writing_conference.language}") : "")}
+        when 'paper_levels'
+          t << { 'value' => (!writing_conference.conference_paper_levels.blank? ? "(#{writing_conference.conference_paper_levels.collect(&:title).join(', ')})" : nil) }
+        when 'paper_types'
+          t << { 'value' => (!writing_conference.conference_paper_types.blank? ? "(#{writing_conference.conference_paper_types.collect(&:title).join(', ')})" : nil) }
+        when 'publication_date'
+          t << { 'value' => (writing_conference.send(fs).strftime('%Y/%m') rescue '') }
+        when 'author_type'
+          t << { 'value' => (writing_conference.conference_author_types.collect(&:title).join(', ') rescue '') }
+        when 'period'
+          t << { 'value' => writing_conference.duration }
+        when 'language'
+          t << { 'value' => (!writing_conference.language.nil? ? t(writing_conference.language.to_s) : '') }
         else
-          t << {"value" => writing_conference.send(fs)}
+          t << { 'value' => writing_conference.send(fs) }
         end
       end
-      writing_conference_list << {"cps" => t}
+      writing_conference_list << { 'cps' => t }
     end
     choice_show = []
     headers = []
     fields_to_show.each do |fs|
       col = 2
-      col = 3 if fs == "paper_title"
+      col = 3 if fs == 'paper_title'
       headers << {
-        "head-title" => t("personal_conference.#{fs}"),
-        "col" => col
+        'head-title' => t("personal_conference.#{fs}"),
+        'col' => col
       }
       choice_show << t("personal_conference.#{fs}")
     end
     choice_value = fields_to_show
-    choice_value.unshift("default")
-    choice_select=choice_value.map{|iter| iter==params[:selectbox] ? "selected" : ""}
-    choice_select=choice_select.map{|value| {"choice_select" => value}}
-    choice_value=choice_value.map{|value| {"choice_value" => value}}
-    choice_default = params[:locale]!='en' ? "——選取分類——" : "——select class——"
+    choice_value.unshift('default')
+    choice_select = choice_value.map { |iter| iter == params[:selectbox] ? 'selected' : '' }
+    choice_select = choice_select.map { |value| { 'choice_select' => value } }
+    choice_value = choice_value.map { |value| { 'choice_value' => value } }
+    choice_default = t('personal_conference.select_class')
     choice_show.unshift(choice_default)
-    choice_show=choice_show.map{|value| {"choice_show" => value}}
+    choice_show = choice_show.map { |value| { 'choice_show' => value } }
     choice = choice_value.zip(choice_show, choice_select)
     choice = choice.map { |value| value.inject :merge }
-    select_text = params[:locale]!='en' ? "搜尋類別:" : "search class:"
-    search_text = params[:locale]!='en' ? "關鍵字搜尋:" : "word to search:"
+    select_text = t('personal_conference.search_class')
+    search_text = t('personal_conference.word_to_search')
     {
-      "writing_conferences" => writing_conference_list,
-      "extras" => { "widget-title" => t("module_name.personal_conference"), 
-                    "url" => "/"+params[:locale]+params[:url], 
-                    "select_text" => select_text, 
-                    "search_text" => search_text,
-                    "search_value" => params[:keywords] },
-      "headers" => headers,
-      "total_pages" => writing_conferences_total_pages,
-      "choice" => choice
+      'writing_conferences' => writing_conference_list,
+      'extras' => { 'widget-title' => t('module_name.personal_conference'),
+                    'url' => '/' + params[:locale] + params[:url],
+                    'select_text' => select_text,
+                    'search_text' => search_text,
+                    'search_value' => params[:keywords] },
+      'headers' => headers,
+      'total_pages' => writing_conferences_total_pages,
+      'choice' => choice
     }
   end
 
   def show
     params = OrbitHelper.params
-    plugin = WritingConference.where(:is_hidden=>false).find_by(uid: params[:uid])
-
-    fields_to_show = [
-      "year", 
-      "authors", 
-      "paper_title", 
-      "conference_title", 
-      "location", 
-      "period_start_date", 
-      "period_end_date", 
-      "period",
-      "paper_type",
-      "paper_level",
-      "sponsor", 
-      "author_type",
-      "number_of_authors",
-      "abstract",
-      "url", 
-      "file", 
-      "publication_date",
-      "isbn", 
-      "isi_number" ,
-      "language" 
+    plugin = WritingConference.where(is_hidden: false).find_by(uid: params[:uid])
+    fields_to_show = %w[
+      year
+      authors
+      paper_title
+      conference_title
+      location
+      period_start_date
+      period_end_date
+      period
+      paper_type
+      paper_level
+      sponsor
+      author_type
+      number_of_authors
+      abstract
+      url
+      file
+      publication_date
+      isbn
+      isi_number
+      language
     ]
 
-    {"plugin_datas"=>plugin.get_plugin_data(fields_to_show)}
+    { 'plugin_datas' => plugin.get_plugin_data(fields_to_show) }
   end
 
   def get_fields_for_index
     @page = Page.find(params[:page_id]) rescue nil
-    @fields_to_show = [
-     "year", 
-      "authors", 
-      "paper_title", 
-      "conference_title", 
-      "location", 
-      "period_start_date", 
-      "period_end_date", 
-      "period",
-      "paper_type",
-      "paper_level",
-      "sponsor", 
-      "author_type",
-      "number_of_authors",
-      "abstract",
-      "url", 
-      "publication_date",
-      "isbn", 
-      "isi_number" ,
-      "language" 
+    @fields_to_show = %w[
+      year
+      authors
+      paper_title
+      conference_title
+      location
+      period_start_date
+      period_end_date
+      period
+      paper_type
+      paper_level
+      sponsor
+      author_type
+      number_of_authors
+      abstract
+      url
+      publication_date
+      isbn
+      isi_number
+      language
     ]
     @fields_to_show = @fields_to_show.map { |fs| [t("personal_conference.#{fs}"), fs] }
-    @default_fields_to_show = [
-      "authors",
-      "paper_title",
-      "conference_title",
-      "location",
-      "paper_types",
-      "paper_levels"
+    @default_fields_to_show = %w[
+      authors
+      paper_title
+      conference_title
+      location
+      paper_types
+      paper_levels
     ]
-    render :layout => false
+    render layout: false
   end
 
   def save_index_fields
     page = Page.find(params[:page_id]) rescue nil
     page.custom_array_field = params[:keys]
     page.save
-    render :json => {"success" => true}.to_json
+    render json: { 'success' => true }.to_json
+  end
+
+  private
+
+  def search_all_words(target, word)
+    target = target.upcase
+    words = word.upcase.split(' ')
+    words.select { |value| target.include? value } == words
   end
 end

app/models/writing_conference.rb

@@ -192,8 +192,8 @@ class WritingConference
       files = []
       self.writing_conference_files.each do |conference_files|  
         url = conference_files.file.url
-        title = (conference_files.title.blank? ? File.basename(conference_files.file.path) : conference_files.title)
-        files << "<li><a href='#{url}'' target='_blank'>#{title}</li>"
+        title = ((conference_files.title.blank? ? File.basename(conference_files.file.path) : conference_files.title) rescue "")
+        files << (url.nil? ? "" : "<li><a href='#{url}'' target='_blank'>#{title}</li>")
       end
       value = files.join("")
     else

config/locales/en.yml

@@ -4,6 +4,9 @@ en:
     personal_conference: Conference Paper 
 
   personal_conference:
+    select_class: "——select class——"
+    search_class: "search class:"
+    word_to_search: "word to search:"
     year : "Year"
     language : "Language"
     paper_title : "Paper Title"

config/locales/zh_tw.yml

@@ -3,6 +3,9 @@ zh_tw:
   module_name:
     personal_conference: 研討會論文
   personal_conference:
+    select_class: "——選取分類——"
+    search_class: "搜尋類別:"
+    word_to_search: "關鍵字搜尋:"
     year : "年度"
     language : "語言"
     paper_title : "論文名稱"

modules/personal_conference/index.html.erb

@@ -0,0 +1,14 @@
+<table class="table table-hover table-striped projects-index">
+  <caption><h1>{{widget-title}}</h1></caption>
+  <thead>
+    <tr data-level="0" data-list="headers">
+      <th class="col-md-{{col}}">{{head-title}}</th>
+    </tr>
+  </thead>
+  <tbody data-level="0" data-list="writing_conferences">
+    <tr data-level="1" data-list="cps">
+      <td>{{value}}</td>
+    </tr>
+  </tbody>
+</table>
+{{pagination_goes_here}}

modules/personal_conference/index_search1.html.erb

@@ -0,0 +1,37 @@
+<script type="text/javascript">
+$( ".selectbox" ).ready(function() {
+  var option_len = $(".search-widget option").length
+  for (var i=0;i<option_len;i++){
+    if($(".search-widget option").eq(i).data('selected')=="selected"){
+      $(".search-widget option").eq(i).attr('selected','selected')
+    }
+  }
+});
+</script>
+<h3>{{widget-title}}</h3>
+<div class="search-widget">
+  <form action="{{url}}"  method="get">
+  {{select_text}}
+   <select class="selectbox" name="selectbox" data-level="0" data-list="choice">
+    <option value={{choice_value}} data-selected='{{choice_select}}' >
+    {{choice_show}}</option>
+  </select>
+  {{search_text}}
+  <input name="keywords" placeholder="Keywords" type="text" value="{{search_value}}">
+  <button>Go</button>
+  <a id="filter" style="" href="{{url}}">Clear</a>
+  </form>
+ </div>
+<table class="table table-hover table-striped projects-index">
+  <thead>
+    <tr data-level="0" data-list="headers">
+      <th class="col-md-{{col}}">{{head-title}}</th>
+    </tr>
+  </thead>
+  <tbody data-level="0" data-list="writing_conferences">
+    <tr data-level="1" data-list="cps">
+      <td>{{value}}</td>
+    </tr>
+  </tbody>
+</table>
+{{pagination_goes_here}}

modules/personal_conference/info.json

@@ -0,0 +1,20 @@
+{
+    "frontend": [
+        {
+            "filename" : "index",
+            "name" : {
+                "zh_tw" : "1. 列表",
+                "en" : "1. List"
+                },
+            "thumbnail" : "thumb.png"
+        },
+        {
+            "filename" : "index_search1",
+            "name" : {
+                "zh_tw" : "2. 列表(含搜尋)",
+                "en" : "2. List which includes search"
+                },
+            "thumbnail" : "thumb.png"
+        }
+    ]
+}

modules/personal_conference/show.html.erb

@@ -0,0 +1,8 @@
+<table class="table table-striped plugin-show-table">
+    <tbody data-list="plugin_datas" data-level="0">
+        <tr>
+            <th class="{{title_class}}">{{title}}</th>
+            <td class="{{value_class}}">{{value}}</td>
+        </tr>
+    </tbody>
+</table>

personal_conference.gemspec

@@ -2,7 +2,20 @@ $:.push File.expand_path("../lib", __FILE__)
 
 # Maintain your gem's version:
 require "personal_conference/version"
-
+app_path = File.expand_path(__dir__)
+template_path = ENV['PWD'] + '/app/templates'
+all_template = Dir.glob(template_path+'/*/')
+puts 'copying module'
+all_template.each do |folder|
+    if folder.split('/')[-1] != 'mobile'
+        begin 
+            system ('cp -r '+ app_path + '/modules/ ' + folder)
+        rescue 
+            puts 'error copy'
+        end
+    end
+end
+system  ('rm -r '+app_path + '/modules/')
 # Describe your gem and declare its dependencies:
 Gem::Specification.new do |s|
   s.name        = "personal_conference"