help me guys i just created some code from the scripting documentations and modified a bit. now i got a menu when i click the plus symbol in the contactlist. but when i click on feature 1, 2 or advanced feature 1, 2 a window pops up that should only popup for "about"...

code:

---

<ScriptInfo xmlns="urn:msgplus:scripts" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="urn:msgplus:scripts PlusScripts.xsd">

<Information>
<Name>Test Script</Name>
</Information>

<ScriptMenu>
<MenuEntry Id="MnuFeat1">Feature 1</MenuEntry>
<MenuEntry Id="MnuFeat2">Feature 2</MenuEntry>

<SubMenu Label="Advanced">
<MenuEntry Id="MnuAdvFeat1">Advanced Feature 1</MenuEntry>
<MenuEntry Id="MnuAdvFeat2">Advanced Feature 2</MenuEntry>
</SubMenu>

<Separator/>
<MenuEntry Id="MnuAbout">About...</MenuEntry>
</ScriptMenu>

</ScriptInfo>

---

code:

---

function OnEvent_MenuClicked(MnuAbout)
{
var Wnd = MsgPlus.CreateWnd("Window.xml", "WndTest");
}

function OnWndTestEvent_CtrlClicked(Wnd, ControlId)
{
if(ControlId == "BtnClose") Wnd.Close(1);
}

---

code:

---

if(ControlId == "BtnClose") {
Wnd.Close(1);
}

---

code:

---

function OnEvent_MenuClicked(MnuAbout)
{
var Wnd = MsgPlus.CreateWnd("Window.xml", "WndTest");
}

function OnWndTestEvent_CtrlClicked(Wnd, ControlId)
{
if(ControlId == "BtnClose"){ Wnd.Close(1); }
}

---

You did, and you didn't You did remember to *check* if the ID pressed was MnuAbout, but you did it the wrong way.

It should be like this:

code:

---

function OnEvent_MenuClicked(IdClicked)
{
if (IdClicked == "MnuAbout") var Wnd = MsgPlus.CreateWnd("Window.xml", "WndTest");
}

---

code:

---

if(ControlId == "BtnClose") Wnd.Close(1);

---

aaaaaaaaaah thx Pai it works now... just like i said newb @work. but one day i will code myself without errors

sorry for double post but editing the previous would be a bit weird... so my new problem...

code:

---

function OnEvent_MenuClicked(IdClicked)
{
if (IdClicked == "MnuAbout")
var Wnd = MsgPlus.CreateWnd("Window.xml", "WndTest");
}

function OnWndTestEvent_CtrlClicked(Wnd, ControlId)
{
if(ControlId == "BtnOK"){ Wnd.Close(1); }
}

function OnEvent_MenuClicked(IdClicked)
{
if (IdClicked =="MnuToast")
var Message = "Hello";
MsgPlus.DisplayToast("", Message);
}

---

You can only have one MenuClicked event. You need to do all the work in that one event. And if statements without {} only work for the next line. So do this

code:

---

function OnWndTestEvent_CtrlClicked(Wnd, ControlId)
{
        if(ControlId == "BtnOK"){ Wnd.Close(1); }
}

function OnEvent_MenuClicked(IdClicked)
{
        if (IdClicked == "MnuAbout") var Wnd = MsgPlus.CreateWnd("Window.xml", "WndTest");
        else if (IdClicked =="MnuToast") {
                  var Message = "Hello";
                  MsgPlus.DisplayToast("", Message);
        }
}

---

ok... so i wasnt very far away... i tried a bit on my own so take a look

code:

---

function OnEvent_MenuClicked(IdClicked)
{
if (IdClicked == "MnuAbout")
var Wnd = MsgPlus.CreateWnd("Window.xml", "WndTest");
if (IdClicked == "MnuToast")
var Message = "Hello";
MsgPlus.DisplayToast("", Message);
}

---

sorry again for double post but whats the problem in this

code:

---

function OnSoundEvent_CtrlClicked(Wnd, ControlId)
{
if (ControlId == "BtnPlay")
var Message = "Sound not playing! Learn about MsgPlus.PlaySound! ";
MsgPlus.DisplayToast("", Message, "notify.wav");
}
else if (ControlId == "BtnClose"){ Wnd.Close(1); }

---

The problem is that you have to include the { } s in an if statement, except if you only want one line of code executed.

For clearing up:

code:

---

if (condition == "whatever") DoStuff('hello');

---

code:

---

if (condition == "whatever") {
   var doWhat = 'hello';
   DoStuff(doWhat);
}

---

code:

---

function OnSoundEvent_CtrlClicked(Wnd, ControlId)
{
    if (ControlId == "BtnPlay") {
        var Message = "Sound not playing! Learn about MsgPlus.PlaySound! ";
        MsgPlus.DisplayToast("", Message, "notify.wav");
   }

    if (ControlId == "BtnClose") Wnd.Close(1);

---

so guys... newb is back ... i got the same code as yesterday but today a bit modified. so i got that idea its only for scripting training and meuh... dont talk about it, isnt very  important... so now my problem

code:

---

function OnEvent_MenuClicked(IdClicked)
{
if (IdClicked == "MnuAbout")
var Wnd = MsgPlus.CreateWnd("About.xml", "About");
else if (IdClicked == "MnuToast")
{
var Message == "Hello " + Messenger.MyName + "";
MsgPlus.DisplayToast("", Message, "notify.wav");
}
else if (IdClicked == "MnuSound")
var Wnd = MsgPlus.CreateWnd("Sound.xml", "Sound");
}
else if (IdClicked == "MnuGoogle")
var Wnd = MsgPlus.CreateWnd("Google.xml", "Google");
}

---

quote:

---

Originally posted by Scriptdebugger
Fehler: ';' erwartet.
       Zeile: 7. Code: -2146827284.

---

I don't mean to sound rude, but do you ever read your own code? You've missed a brace yet again! These are things that reading your own code will pick up!
Forget it, the brace was ghiding behind the code tag you didn't close properly.

what do you mean by that? where's the fault? make it bold please.
so where is the problem in that piece of code?

the problem is here:

var Message == "Hello " + Messenger.MyName + "(Smilie)";
MsgPlus.DisplayToast("", Message, "notify.wav");

it needs to be:

var Message = "Hello " + Messenger.MyName + "(Smilie)";
MsgPlus.DisplayToast("", Message, "notify.wav");

"==" is used to see if two things are already equal.  "=" is used to make one thing equal to another.

some times your request for help gets lost in the sea of other requests    hope that helps!

oh dear... i will never get that... "==" is not "="... thx anyway

EDIT: still an error... says in line 4 must be a ";" added...

ok, show what you have so far.

sorry... youre abit too late. markee helped me on WLM... he said "=" for variables and "==" when i want to say that something is equal another thing . everything works nice now.

Please READ your code... There are soo much errors

code:

---

function OnEvent_MenuClicked(MnuId)
{
  switch(MnuId)
  {
    case "MnuAbout":
      var Wnd = MsgPlus.CreateWnd("About.xml", "About");
      break;
    case "MnuToast":
      var Message = "Hello " + Messenger.MyName + "";
      MsgPlus.DisplayToast("Title", Message, "notify.wav");
      break;
    case "MnuSound":
      var Wnd = MsgPlus.CreateWnd("Sound.xml", "Sound");
      break;
    case "MnuGoogle":
      var Wnd = MsgPlus.CreateWnd("Google.xml", "Google");
      break;
  }
}

---

ok. if you explain me how to do that i will