I have no experience with Jscript so I'm struggling quite a bit, and would love some help

Basically I'm making a window with 3 listviews (I would love for it to be one with three columns, but I just cant get it to work), they display different elements of a certain XML file (I generate it via other means).

Anyways, here is what I made so far, I downloaded a few scripts and tried to copy what they where doing, but it didint work out as planned hehe.

code:

---

function OnEvent_Initialize(MessengerStart)
{
}

function OnEvent_Uninitialize(MessengerExit)
{
}

var xmlDoc = new ActiveXObject("Microsoft.XMLDOM");
function loadXML(xmlFile)
{
  xmlDoc.async="false";
  xmlDoc.onreadystatechange=verify;
  xmlDoc.load(xmlFile);
  xmlObj=xmlDoc.documentElement;
}

function OnEvent_MenuClicked (MenuItemId, Location, OriginWnd)
{
    // Captures when the user clicks on the 'WLMBb' menu
    if (MenuItemId=="Manage") fillTable();
}

function onTareasEvent_CtrlClicked (PlusWnd, ControlId)
{
    if (ControlId == "Refresh")
    {
        fillTable();
    }   
}

function fillTable()
{
    // Opens the window and loads the tasks from the XML file.
    var Manage = MsgPlus.CreateWnd("Interface.xml", "Manage");

    loadXML('C:\\Program Files\\Messenger Plus! Live\\Scripts\\wlmbb\\tasks.xml');
    TaskList = xmlObj.getElementsByTagName("tasks")[0].getElementsByTagName("task");
    IntTaskList = TaskList.lenght
   
    for (IntCurrentTask = 0; IntCurrentTask < IntTaskList; IntCurrentTask++)
    {
        CurrentTask = xmlObj.getElementByTagName("tasks")[0].getElementsByTagName("task")[IntCurrentTask];
        IntId = 0;
        Manage.LstView_AddItem("ListViewName", CurrentTask.getElementsByTagName("name")[0].text, IntId++);
        Manage.LstView_AddItem("ListViewExpiration", CurrentTask.getElementsByTagName("name")[0].text, IntId++);
        Manage.LstView_AddItem("ListViewCourse", CurrentTask.getElementsByTagName("name")[0].text, IntId++);
    }
}

---

code:

---

<?xml version="1.0" encoding='UTF-8'?>
<tasks>
    <task id="1">
        <name>nameofthistask1</name>
        <expiration>may 21 2010</expiration>
        <course>nameofthiscourse1</course>
    </task>
    <task id="2">
        <name>somethingelse</name>
        <expiration>may 22 2010</expiration>
        <course>Cooking class</course>
    </task>
    <task id="3">
        <name>listview</name>
        <expiration>may 23 2010</expiration>
        <course>scripting</course>
    </task>
    <task id="4">
        <name>WLMBb</name>
        <expiration>mayo 24 2010</expiration>
        <course>coolstuff</course>
    </task>
</tasks>

---

Here is some source code that we use in Screenshot Sender. If it doesn't make sense let me know.

js code:

---

function EnumControls(File, pPlusWnd) {
    _debug.getfuncname(arguments);
    // Load the XML from the specified file
    var XML = new ActiveXObject('MSXML.DOMDocument');
    XML.load(File);
    // Loop through all of the controls
    var Controls = XML.selectNodes('/Interfaces/Window[@Id=\'' + pPlusWnd.WindowId + '\']/Controls/Control');
    for (i=0; i<Controls.length; i ++) {
        var Id = Controls[i].getAttribute('Id');
        var Type = Controls[i].getAttribute('xsi:type');
        // Controls starting with _ are options but not stired in the registry (ie Debug info is in the ScriptInfo.xml)
        if (Id.charAt(0) !== '_') {
            // Get the value depending on the control's type
            objControls[Id] = {};
            objControls[Id].XsiType = Type;
            objControls[Id].Value = pPlusWnd_GetControlvalue(pPlusWnd, Id, Type);
        }
    }
}

---

Hmm, seems far simpler that way...

So my script should go something like this:

code:

---

var Tasks = XML.selectNodes('/Tasks/Task');

for (i=0; i<Tasks.length; i ++)
{
Manage.LstView_AddItem("ListViewName", Tasks[i].methodtogetvalueofname, IntId++);

...
}

---

You can use selectSingleNode(strPattern) to select the first descendant node which matches a certain pattern, like a tag selector.

js code:

---

var Tasks = XML.selectNodes("/Tasks/Task");

for (i=0; i<Tasks.length; i ++) {
    var TaskNode = Tasks[i];
    var NameNode = TaskNode.selectSingleNode("Name");
    Manage.LstView_AddItem("ListViewName", NameNode.text, IntId++);
}

---

Just did a quick read of XPath and it seems quite simple, but I'll stick to the selectSingleNode for now.

I'm currently testing the script but I think that I'm failing to load the actual tasks.xml file (its located inside my script folder)

code:

---

    var XML = new ActiveXObject('MSXML.DOMDocument');
    XML.load(MsgPlus.ScriptFilesPath + '/tasks.xml');

---

Could you provide us with the details from the script debugger window?

Anyway, I think there are two possibilities for this problem:

1. The ActiveXObject couldn't be created.
   I had this before, for some reason my Windows installation couldn't recognize Microsoft.XMLDOM any more. After some research, I found that MSXML2.DOMDocument still worked, but then again that didn't work on other computers. I decided to implement both of them in a try...catch block so that it would always find a working ActiveXObject.
   So, instead of using new ActiveXObject('MSXML.DOMDocument');, I now call CreateXML() after defining it as followed:
   

   js code:

   ---

   //Creates a new XMLDOM instance
   function CreateXML() {
       try {
           var xml = new ActiveXObject("Microsoft.XMLDOM");
       } catch(e) { try {
           var xml = new ActiveXObject("MSXML2.DOMDocument");
       } catch(e) { return; } }
       xml.async = false;
       return xml;
   }

   ---

2. The path may be faulty.
   I see that you're using a slash in your path, try a backslash instead. Of course, in JScript you have to escape a backslash with a backslash, so that line should look something like this:
   

   js code:

   ---

   XML.load(MsgPlus.ScriptFilesPath + '\\tasks.xml');

   ---

3. The file might still be loading when you try to access it.
   This can be easily fixed by setting async to false, forcing the XML DOM object to load the XML file synchronously. As you can see above, my function does this automatically since I don't use asynchronous XML loading very often in my scripts.

Hint: these forums can do syntax highlighting by wrapping your code in [code=js] ... [/code] tags (for JScript, that is), it makes your code easier to read for others.

Tried everything suggested, the script seems to just ignore my function calls =/


Here is my entire code:

js code:

---

function OnEvent_Initialize(MessengerStart)
{
}

function OnEvent_Uninitialize(MessengerExit)
{
}

function OnEvent_MenuClicked (MenuItemId, Location, OriginWnd)
{
    // Captures when the user clicks on the 'WLMBb' menu
    if (MenuItemId=="Manage") fillTable();
}

function CreateXML() {
    try {
        var xml = new ActiveXObject("Microsoft.XMLDOM");
    } catch(e) { try {
        var xml = new ActiveXObject("MSXML2.DOMDocument");
    } catch(e) { return; } }
    xml.async = false;
    return xml;
}

function onManageEvent_CtrlClicked (PlusWnd, ControlId)
{
    if (ControlId == "Refresh")
    {
        fillTable();   
    }   
}

function fillTable()
{
    // Opens the window and loads the tasks from the XML file.
    var Manage = MsgPlus.CreateWnd("Interface.xml", "Manage");

    var XML = CreateXML();
        XML.load('tasks.xml');
    var Tasks = XML.selectNodes('/Tasks/Task');
    IntTaskList = Tasks.lenght
    IntId = 0;
    for (i=1; i<Tasks.length; i ++)
    {
       
        var TaskNode = Tasks[i];
        var NameNode = TaskNode.selectSingleNode("Name");
        var ExpiNode = TaskNode.selectSingleNode("Expiration");
        var CrseNode = TaskNode.selectSingleNode("Course");
        Manage.LstView_AddItem("ListViewName", NameNode.text, IntId++);
        Manage.LstView_AddItem("ListViewExpiration", ExpiNode.text, IntId++);
        Manage.LstView_AddItem("ListViewCourse", CrseNode.text, IntId++);   
    }
}

---

code:

---

Script is starting
Script is now loaded and ready
Function called: OnEvent_Initialize
Function called: OnEvent_MenuClicked
Function called: OnManageEvent_CtrlClicked

---

How many tasks are in the tasks.xml? I can already see one issue, namely that you're starting from index 1 for your loop. In JScript (and actually all languages), arrays and collections start at index 0 so you're skipping the first element with that loop. I believe that you have just one task in the XML for testing this and therefore the for-loop immediately exits when it checks the condition.

Also, you don't need those IntTaskList and IntId variables. The length of the tasks list is only used once (for the loop condition) and the identifier is unnecessary and even incorrectly used. You're using it to determine the position of the inserted list-view item, however you're actually incrementing it three times for each item. For the first list-view, you increment the position, but you're also doing this for the second and third list-view.

The positions you expect:

code:

---

Task index   ListViewName   ListViewExpiration   ListViewCourse
   0            0              0                    0
   1            1              1                    1
   2            2              2                    2
   3            3              3                    3

---

code:

---

Task index   ListViewName   ListViewExpiration   ListViewCourse
   0            0              1                    2
   1            3              4                    5
   2            6              7                    8
   3            9             10                   11

---

js code:

---

    var IntId = 0;
    for (var i=0; i<Tasks.length; i++)
    {
        var TaskNode = Tasks[i];
        var NameNode = TaskNode.selectSingleNode("Name");
        var ExpiNode = TaskNode.selectSingleNode("Expiration");
        var CrseNode = TaskNode.selectSingleNode("Course");
>>>        Manage.LstView_AddItem("ListViewName", NameNode.text, IntId);<<<
>>>        Manage.LstView_AddItem("ListViewExpiration", ExpiNode.text, IntId);<<<
>>>        Manage.LstView_AddItem("ListViewCourse", CrseNode.text, IntId);<<<
>>>        IntId++;<<<
    }

---

• Since you're also using this function as refresh function when the window is already open, it is better to move the window creation outside the fillTable function. Otherwise, you'll be opening new windows every time you click Refresh.
  
• Also for your refreshing, you need to clear the list-views before re-adding the items. The fastest way to do this is by sending LVM_DELETEALLITEMS to the three controls.
  
• The for-loop can be optimized by storing the length of the collection and by using pre-incrementation instead of post-incrementation. This is most of the time not very important, but it's good practice to use efficient loops.

js code:

---

function OnEvent_MenuClicked(MenuItemId, Location, OriginWnd)
{
    // Captures when the user clicks on the 'WLMBb' menu
    if (MenuItemId === "Manage")
    {
        // Open the window
        var Manage = MsgPlus.CreateWnd("Interface.xml", "Manage");
        // Refresh the table
        fillTable();
    }
}

function CreateXML()
{
    try    {
        var xml = new ActiveXObject("Microsoft.XMLDOM");
    } catch(e) { try {
        var xml = new ActiveXObject("MSXML2.DOMDocument");
    } catch(e) { return; } }
    xml.async = false;
    return xml;
}

function OnManageEvent_CtrlClicked(PlusWnd, ControlId)
{
    if (ControlId == "Refresh")
    {
        // Refresh the table
        fillTable();
    } 
}

function fillTable()
{
    // Clear all list-views before adding new items
    var LVM_DELETEALLITEMS = 0x1009; // = LVM_FIRST + 9
    Manage.SendControlMessage("ListViewName", LVM_DELETEALLITEMS, 0, 0);
    Manage.SendControlMessage("ListViewExpiration", LVM_DELETEALLITEMS, 0, 0);
    Manage.SendControlMessage("ListViewCourse", LVM_DELETEALLITEMS, 0, 0);

    // Load the tasks from the XML file
    var XML = CreateXML();
    XML.load('tasks.xml');
    var Tasks = XML.selectNodes('/Tasks/Task');
    // Optimized the loop: start at 0, store length in len and use pre-increment
    for (var i=0, len=Tasks.length; i<len; ++i)
    {
        var TaskNode = Tasks[i];
        var NameNode = TaskNode.selectSingleNode("Name");
        var ExpiNode = TaskNode.selectSingleNode("Expiration");
        var CrseNode = TaskNode.selectSingleNode("Course");
        // No need to pass item position as parameter, Plus! will handle that
        Manage.LstView_AddItem("ListViewName", NameNode.text);
        Manage.LstView_AddItem("ListViewExpiration", ExpiNode.text);
        Manage.LstView_AddItem("ListViewCourse", CrseNode.text); 
    }
}

---

js code:

---

for(var i = 0, TaskNode; TaskNode = Tasks[i++]; ) {
    // ...
}

---

js code:

---

for(var i = -1, TaskNode; TaskNode = Tasks[++i]; ) {
    // ...
}

---