Forum Discussion

royd's avatar
royd
Regular Contributor
7 years ago

Global variable troubles (Calling Variables Declared in Another Unit) in javascript

I have successfully used "//USEUNIT" method of calling functions and variables in the past.  I am trying to use "module.exports" method and can not figure out how  to make it work. Here is what I have -

 

The source unit:

 

//Patinet_Information
var nameOfPatient;
function patientStat(){
   ...
  nameOfPatient = page.FindChild("idStr", "patientName", 10).contentText;
   ...
 }

 module.exports.nameOfPatient;

 

 

The destination unit:

 

var patientInfo = require("Patinet_Information");
function createPatient(){
    ...
  let pName = page.FindChildByXPath("//input[@id='userName']");;
  pName.SetText(nameOfPatient);
}

I have also tried 

  pName.SetText(patientInfo.nameOfPatient);

 

 

did not work ("nameOfPatient is undefined").

 

Thanks in advance for any help.

 

Dave

  • cunderw's avatar
    cunderw
    7 years ago

    Ah, I forgot the Javascript scope can be a little weird. You won't actually be able to reference that var directly without it being a const. 

     

    So you could approach it two different ways:

     

    let fooString = ""

function setFooString(){
 
 fooString = "Test";
 
}

function getFooString() {
  
  return fooString;
}

module.exports.setFooString = setFooString;
module.exports.getFooString = getFooString;
    // bar file

let foo = require("foo");

function barFunction() { 
  foo.setFooString();
  ShowMessage(foo.getFooString());
}

    Or you could do a const object like this:

     

    // foo file
const info = {
  fooString : ""
}

function setFooString(){
 
 info.fooString = "Test";
 
}

module.exports.setFooString = setFooString;
module.exports.info = info;
    // bar file

let foo = require("foo");

function barFunction() { 
  foo.setFooString();
  ShowMessage(foo.info.fooString);
}

    I personally would prefer the second way because it's much easier to expand on what you're acceessing from the first unit.

  • cunderw's avatar
    cunderw
    Community Hero
    7 years ago

    The first thing I see is that 

    module.exports.nameOfPatient;

    should be 

    module.exports.nameOfPatient = nameOfPatient;

    You have to specify what the export is referencing.  

    • royd's avatar
      royd
      Regular Contributor
      7 years ago

      Thank you. I will try that and report it when I get a chance.

    • royd's avatar
      royd
      Regular Contributor
      7 years ago

      Hi cunderw

       

      This morning corrected and tried your suggestion. I also realized that I missed a few things as well, so here is what I did -

      //Patinet_Information
      var nameOfPatient;
      function patientStat(){
       ...
       nameOfPatient = page.FindChild("idStr", "patientName", 10).contentText;
       ...
       }

       //module.exports.patientStat = patientStat;
       module.exports.nameOfPatient = nameOfPatient;

      and ...

       var Patinet_Information = require("Patinet_Information");
      function createPatient(){
       ...
       let pName = page.FindChildByXPath("//input[@id='userName']");;
       pName.SetText(Patinet_Information.nameOfPatient);
      }

        I am still getting nameOfPatinet 'undefined'.

      • cunderw's avatar
        cunderw
        Community Hero
        7 years ago

        Ah, I forgot the Javascript scope can be a little weird. You won't actually be able to reference that var directly without it being a const. 

         

        So you could approach it two different ways:

         

        let fooString = ""

function setFooString(){
 
 fooString = "Test";
 
}

function getFooString() {
  
  return fooString;
}

module.exports.setFooString = setFooString;
module.exports.getFooString = getFooString;
        // bar file

let foo = require("foo");

function barFunction() { 
  foo.setFooString();
  ShowMessage(foo.getFooString());
}

        Or you could do a const object like this:

         

        // foo file
const info = {
  fooString : ""
}

function setFooString(){
 
 info.fooString = "Test";
 
}

module.exports.setFooString = setFooString;
module.exports.info = info;
        // bar file

let foo = require("foo");

function barFunction() { 
  foo.setFooString();
  ShowMessage(foo.info.fooString);
}

        I personally would prefer the second way because it's much easier to expand on what you're acceessing from the first unit.