"Missing semicolon" compile error on a script

[arnoldzwane]

I am getting a "Missing semicolon" compile error on line 10 "If (gen.u_gen_cpu_name == ci.cpu_name){" of my script below: 

//get CIs
var ci = new GlideRecord('cmdb_ci_computer');
ci.query();
while (ci.next()) {
   //get the Generation record for this CI (only one record)
   var gen = new GlideRecord("u_cpu_generation");
   gen.addQuery("ci",ci.sys_id.toString());
   gen.query();
   while (gen.next()){
      If (gen.u_gen_cpu_name == ci.cpu_name){
         ci.u_cpu_generation = gen.u_gen_cpu_generation;
         ci.u_processor_windows_10_capable = gen.u_processor_windows_10_capable;
      }
      ci.setWorkflow(false);
      ci.update();
   }
}

When I put it (even though I don't understand why would it need a semicolon on the 'if" statement) the error goes away but the script doesn't do what I expect it to do.

Could someone please assist. I just need to update the two fields, u_cpu_generation and u_processor_windows_10_capable on the Computer table, with data from the u_cpu_generation table. You are welcome to suggest changes on my script, that will enable me to achieve the desired result - still a learner in scripting!

[vinothkumar]

It looks like, you are referring a custom table in your query named "u_cpu_generation".

But in query, you are using CI field in your add query. Are you sure, is there any field available on that table?

var gen = new GlideRecord("u_cpu_generation");
   gen.addQuery("ci",ci.sys_id.toString());

[arnoldzwane]

Hi Vinothkumar,

I realised that later and changed it to  gen.addQuery("u_gen_cpu_name",ci.cpu_name); with the hope of getting a match on cpu_name as I know for sure that cpu_name on Computer table has a matching record on the custom table. But this didn't do it either...

Kind Regards

Arnold

[Ankur Bawiskar]

Hi Arnold,

The method setWorkflow(false) won't run in client script and will result in error whatever you try to do here if this script is present in client script.

Mark Correct if this solves your issue and also hit Like and Helpful if you find my response worthy based on the impact.
Thanks
Ankur

[arnoldzwane]

Hi Ankur,

I 'm running this as a Scheduled Job. I just tried running it without a setWorkflow(false) method and still no desired result.....

Kind Regards

Arnol

[Shweta KHAJAPUR]

Try with below script,

var ci = new GlideRecord('cmdb_ci_computer');
ci.query();
while (ci.next()) {
   
   var gen = new GlideRecord("u_cpu_generation");
   gen.addQuery("gen.u_gen_cpu_name",ci.cpu_name);
   gen.query();
   if(gen.next()){
      
         ci.u_cpu_generation = gen.u_gen_cpu_generation;
         ci.u_processor_windows_10_capable = gen.u_processor_windows_10_capable;
      
         ci.update();
   }
}