MODExt.Msg.alert error

☆ A M B ☆
450 Posts

jrotering Reply #1, 15 years, 4 months ago

I am trying to follow the CMP tutorial/example; I created my Namespace, Controller file, Action and Menu Object. In my Controller file I’m including a js file like this:

 $modx->regClientStartupScript('/assets/components/widget_hilite/js/hilite.js');
return 'This is my first Custom Manager Page';

My .js file is responsive - when I put a simple "alert()" statement in it, it works. But when I try to use MODExt.Mst.alert, it throws an error.

MODx.Msg.alert('Warning!','You are out of space! We should clear the cache.',function() {
  MODx.clearCache();
},MODx);

"Uncaught TypeError: Object function (A){A=A||{};MODx.Msg.superclass.constructor.call(this,A);this.addEvents({success:true,failure:true,cancel:true});Ext.MessageBox.minWidth=200;} has no method ’alert’"

Am I missing a necessary step for using MODExt?

4,149 Posts

splittingred Reply #2, 15 years, 4 months ago

MODx.msg.alert

with a lowercase ’m’ in msg. JS is case-sensitive, in this sense.

shaun mccormick | bigcommerce mgr of software engineering, former modx co-architect | github | splittingred.com

☆ A M B ☆
450 Posts

jrotering Reply #3, 15 years, 4 months ago

Quote from: splittingred at May 26, 2011, 10:57 AM

MODx.msg.alert
with a lowercase ’m’ in msg. JS is case-sensitive, in this sense.

1. I took that code directly from the MODx.Msg RTFM page (http://rtfm.modx.com/display/revolution20/MODx.Msg)
2. When I change it to lower-case I still get an error, albeit a different one:

Uncaught TypeError: Object #<Object> has no method 'alert'

4,149 Posts

splittingred Reply #4, 15 years, 4 months ago

I’ve fixed the docs. Thanks for catching that!

Where are you putting the code? You’ll need to make sure to do it *after* MODx.msg loads, so at the very least wrap it in a Ext.onReady clause:

Ext.onReady(function() {
    MODx.msg.alert('Warning!','You are out of space! We should clear the cache.',function() {
      MODx.clearCache();
    },MODx);
});

shaun mccormick | bigcommerce mgr of software engineering, former modx co-architect | github | splittingred.com

☆ A M B ☆
450 Posts

jrotering Reply #5, 15 years, 4 months ago

That works. I was just putting it in using $modx->regClientStartupScript() according to the recommendation on the "Scripts and CSS" section on the CMP page (http://rtfm.modx.com/display/revolution20/Custom+Manager+Pages)

I guess just like with jquery, you need to wrap most of it in an onready statement in order to ensure that everything is properly loaded(?)

4,149 Posts

splittingred Reply #6, 15 years, 4 months ago

Quote from: jrotering at May 26, 2011, 11:17 AM

I guess just like with jquery, you need to wrap most of it in an onready statement in order to ensure that everything is properly loaded(?)

Yep.

shaun mccormick | bigcommerce mgr of software engineering, former modx co-architect | github | splittingred.com