nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

Item handler problem

Huan

I registered two item handlers in my init function and explicitly called them in this function.


function init() {
IWEventRegistry.addItemHandler("/cms/language", "onItemChange", languageHandler);
 IWEventRegistry.addItemHandler("/cms/general/type", "onItemChange", typeHandler);

languageHandler(IWDatacapture.getItem("/cms/language"));
  typeHandler(IWDatacapture.getItem("/cms/general/type"));
}

The language handler checks the current language and update an option list called province in either English or French.


function languageHandler(typeItem) {
var lan = typeItem.getOptions()[typeItem.getValue()].value;
var pItem = IWDatacapture.getItem("/cms/location/province");
var p_value = pItem.getOptions()[pItem.getValue()].value;
if (lan == "eng") {
//alert ("pvalue=" + p_value);
    if (p_value == "AB") {
        provinceOptions[0] = new Option("Alberta", "AB", true, true);
    } else {
        provinceOptions[0] = new Option("Alberta", "AB", false, false);
    }
    provinceOptions[1] = new Option("British Columbia", "BC", false, false);
} else if (lan == "fre") {
 ....
} ....
IWDatacapture.getItem ("/cms/location/province").setOptions(provinceOptions);
}

However, it seems that I cannot update the list and meanwhile set the selected one. If I uncomment out the alert, it works, that is, it updates the option list and set the selected one to Alberta. But after I comments out the alert, it never set the selected one to Alberta even though I have the correct province in my DCR.

Sometimes, none of the userscript works and no error message shown.

Find more posts tagged with

Comments

Migrateduser

First of all, how are you defining typeItem and pItem?

Huan

var typeItem = IWDatacapture.getItem("/cms/language");
var pItem = IWDatacapture.getItem("/cms/location/province");

Migrateduser

A couple of things -- although I'm doubtful that this is your issue:

1. It would be nice if you could declare "var provinceOptions = new Array();" since the way your code is scoped, it could get down to your setOptions statement and not have knowledge of what provinceOptions is.

2. I don't know if this is the way your code was pasted on DevNet or not but it looks like you're missing a couple of { }. I understand that you don't need them when you're doing just one statement but you're setting provinceOptions[0] and [1] (i.e., two statements) and you appear to have no braces there.

Other than these, I really can't see anything wrong with your code.

Huan

Thanks for your comments. I have corrected them in my code.

Still the problem is if I put alert, the code after that statement is executed, however, if I remove it, it seems the code is not executed at all. Does userscript need to be synchronized?