Fix check unique Job data by name,city,company

lib/tasks/careerbuilder.rb

@@ -83,10 +83,12 @@ class Careerbuilder
 
 
   def detail(doc, _link)
   def detail(doc, _link)
     # Company Information
     # Company Information
+    company = Company.new
     company_name = doc.xpath("//div[@class='tit_company']").text.strip # Company name
     company_name = doc.xpath("//div[@class='tit_company']").text.strip # Company name
-    company_address = doc.xpath("//div[@class='box1Detail']/p[@class='TitleDetailNew']/label[@itemprop='address']/label[@itemprop='addressLocality']").text.strip # Company Address
-    company_description = doc.xpath("//div[@class='desc_company content_fck']").text.strip # Company description
-    Company.new(name: company_name, address: company_address, description: company_description).save if Company.where(name: company_name).blank?
+    company.name = company_name
+    company.address = doc.xpath("//div[@class='box1Detail']/p[@class='TitleDetailNew']/label[@itemprop='address']/label[@itemprop='addressLocality']").text.strip # Company Address
+    company.description = doc.xpath("//div[@class='desc_company content_fck']").text.strip # Company description
+    company.save if Company.where(name: company_name).blank?
 
 
     # Job Information
     # Job Information
     job_name = doc.xpath("//div[@class='LeftJobCB']/div[@class='top-job']/div[@class='top-job-info']/h1").text.strip # Job name
     job_name = doc.xpath("//div[@class='LeftJobCB']/div[@class='top-job']/div[@class='top-job-info']/h1").text.strip # Job name
@@ -105,14 +107,23 @@ class Careerbuilder
 
 
     job_expiry_date = doc.xpath("//ul[@class='DetailJobNew']/li/p[span/text()='Hết hạn nộp: ']/text()").to_s
     job_expiry_date = doc.xpath("//ul[@class='DetailJobNew']/li/p[span/text()='Hết hạn nộp: ']/text()").to_s
 
 
-    Job.new(name: job_name, description: job_description,
-            salary: job_salary,
-            city: City.find_by_name(job_location),
-            level: job_level, experience: job_experience, status: 0,
-            expiry_date: job_expiry_date.to_datetime).save
+    city = City.find_by_name(job_location)
+
+    job = Job.new
+    job.name = job_name
+    job.description = job_description
+    job.salary = job_salary
+    job.city = city
+    job.company = company
+    job.level = job_level
+    job.experience = job_experience
+    job.status = 0
+    job.expiry_date = job_expiry_date.to_datetime
+    job.save if Job.where(name: job_name, city: city, company: company).blank?
 
 
     job_category.split(',').each do |category|
     job_category.split(',').each do |category|
-      JobCategory.new(job: Job.find_by_name(job_name), category: Category.find_by_name(category)).save
+      category = Category.find_by_name(category)
+      JobCategory.new(job: job, category: category).save if JobCategory.where(job: job, category: category).blank?
     end
     end
   end
   end