correct model method
This commit is contained in:
		
							parent
							
								
									228dd1b767
								
							
						
					
					
						commit
						dc68f40cb4
					
				|  | @ -42,7 +42,6 @@ class WritingConference | ||||||
|   validates :paper_title, :at_least_one => true |   validates :paper_title, :at_least_one => true | ||||||
|   before_validation :add_http |   before_validation :add_http | ||||||
|   after_save :save_writing_conference_files |   after_save :save_writing_conference_files | ||||||
|   before_save :update_co_author |  | ||||||
|   validates :url, :format => /^(http|https):\/\/(([a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5})|((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?))(:[0-9]{1,5})?(\/.*)?/i, :unless => Proc.new{self.url.blank?} |   validates :url, :format => /^(http|https):\/\/(([a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5})|((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?))(:[0-9]{1,5})?(\/.*)?/i, :unless => Proc.new{self.url.blank?} | ||||||
| 
 | 
 | ||||||
|   attr_reader :author_tokens |   attr_reader :author_tokens | ||||||
|  | @ -64,19 +63,6 @@ class WritingConference | ||||||
|     self.conference_co_author_ids = authors_ids |     self.conference_co_author_ids = authors_ids | ||||||
|   end |   end | ||||||
| 
 | 
 | ||||||
|   def update_co_author |  | ||||||
|     if new_record? |  | ||||||
|       current_user = self.create_user_id |  | ||||||
|     else |  | ||||||
|       current_user = self.update_user_id |  | ||||||
|     end |  | ||||||
| 
 |  | ||||||
|     self.conference_co_author_ids.each do |id| |  | ||||||
|       co_author = ConferenceCoAuthor.find(id) |  | ||||||
|       co_author.update_attribute(:name_id, current_user) |  | ||||||
|     end |  | ||||||
|   end |  | ||||||
| 
 |  | ||||||
|   def self.search( category_id = nil ) |   def self.search( category_id = nil ) | ||||||
|     if category_id.to_s.size > 0 |     if category_id.to_s.size > 0 | ||||||
|       find(:all, :conditions => {writing_conference_category_id: category_id}).desc( :is_top, :title ) |       find(:all, :conditions => {writing_conference_category_id: category_id}).desc( :is_top, :title ) | ||||||
|  |  | ||||||
|  | @ -36,8 +36,6 @@ class WritingJournal | ||||||
|   accepts_nested_attributes_for :writing_journal_files, :allow_destroy => true |   accepts_nested_attributes_for :writing_journal_files, :allow_destroy => true | ||||||
|   after_save :save_writing_journal_files |   after_save :save_writing_journal_files | ||||||
|   before_validation :add_http |   before_validation :add_http | ||||||
|   #before_save :save_co_author |  | ||||||
|   before_save :update_co_author |  | ||||||
|   validates :paper_title, :at_least_one => true |   validates :paper_title, :at_least_one => true | ||||||
|   validates :url, :format => /^(http|https):\/\/(([a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5})|((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?))(:[0-9]{1,5})?(\/.*)?/i, :unless => Proc.new{self.url.blank?} |   validates :url, :format => /^(http|https):\/\/(([a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5})|((25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?))(:[0-9]{1,5})?(\/.*)?/i, :unless => Proc.new{self.url.blank?} | ||||||
| 
 | 
 | ||||||
|  | @ -65,35 +63,6 @@ class WritingJournal | ||||||
|     self.journal_co_author_ids = authors_ids |     self.journal_co_author_ids = authors_ids | ||||||
|   end |   end | ||||||
| 
 | 
 | ||||||
|   def update_co_author |  | ||||||
|     if new_record? |  | ||||||
|       current_user = self.create_user_id |  | ||||||
|     else |  | ||||||
|       current_user = self.update_user_id |  | ||||||
|     end |  | ||||||
| 
 |  | ||||||
|     self.journal_co_author_ids.each do |id| |  | ||||||
|       co_author = JournalCoAuthor.find(id) |  | ||||||
|       co_author.update_attribute(:name_id, current_user) |  | ||||||
|     end |  | ||||||
|   end |  | ||||||
|   # old method |  | ||||||
|   #def save_co_author |  | ||||||
|   #  authors_list = authors.split(",").map{|n| n.strip.gsub(/\s+/," ")}.reject{|n| n.empty? } |  | ||||||
|   #  if new_record? |  | ||||||
|   #    current_user = create_user_id |  | ||||||
|   #  else |  | ||||||
|   #    current_user = update_user_id |  | ||||||
|   #  end |  | ||||||
| 
 |  | ||||||
|   #  authors_db = JournalCoAuthor.where(:name_id => current_user).map(&:co_author) |  | ||||||
|   #  authors_list.delete(User.find(current_user).name) |  | ||||||
|   #  authors_list = authors_list.delete_if{|author| authors_db.include?author} |  | ||||||
|   #  authors_list.each do |author| |  | ||||||
|   #    JournalCoAuthor.new(:co_author => author, :name_id => current_user).save |  | ||||||
|   #  end |  | ||||||
|   #end |  | ||||||
| 
 |  | ||||||
|   def self.search( category_id = nil ) |   def self.search( category_id = nil ) | ||||||
|     if category_id.to_s.size > 0 |     if category_id.to_s.size > 0 | ||||||
|       find(:all, :conditions => {writing_journal_category_id: category_id}).desc( :is_top, :title ) |       find(:all, :conditions => {writing_journal_category_id: category_id}).desc( :is_top, :title ) | ||||||
|  |  | ||||||
|  | @ -13,10 +13,8 @@ end | ||||||
|   FactoryGirl.create(:journal_co_author_relation) |   FactoryGirl.create(:journal_co_author_relation) | ||||||
| end | end | ||||||
| 
 | 
 | ||||||
| 4.times do | 50.times do | ||||||
|   50.times do |  | ||||||
|   FactoryGirl.create(:writing_journal) |   FactoryGirl.create(:writing_journal) | ||||||
|   end |  | ||||||
| end | end | ||||||
| 
 | 
 | ||||||
| 50.times do | 50.times do | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue