MODx Recursive Parser - Beta 2a | MODX Community Forums

MODX Staff
12,272 Posts

Ryan Thrash Reply #91, 20 years, 2 months ago

I think pagination would be quite handy. For instance, if someone wanted to work with a 1000 row table, that’d be a bit nasty without it. wink

Ryan Thrash, MODX Co-Founder
Follow me on Twitter at @rthrash or catch my occasional unofficial thoughts at thrash.me

1,732 Posts

xwisdom Reply #92, 20 years, 2 months ago

Quote from: rthrash at Jul 31, 2006, 05:10 PM

I think pagination would be quite handy. For instance, if someone wanted to work with a 1000 row table, that’d be a bit nasty without it.

Yeah, I agree with you 100%. I’ll add pagination in Beta 3.

xWisdom
www.xwisdomhtml.com
The fear of the Lord is the beginning of wisdom:
MODx Co-Founder - Create and do more with less.

1,732 Posts

xwisdom Reply #93, 20 years, 1 month ago

Hi Everyone,

Just checking in.

Has anyone found any bugs, performance issues or you have some suggestions, fixes or tweaks?

(If not then I guess it can be declared stable - lol)

xWisdom
www.xwisdomhtml.com
The fear of the Lord is the beginning of wisdom:
MODx Co-Founder - Create and do more with less.

398 Posts

bS Reply #94, 20 years, 1 month ago

If i copy the parser into the latest 0.9.5 version (from public SVN - retrash trunk) and run something in the frontend i get a couple of warnings & errors. I’ll post the errors messages later today (i’m not home right now and unable to check my local setup from here.)

Armand Pondman
MODx Coding Team
:: Jot :: PHx

1,732 Posts

xwisdom Reply #95, 20 years, 1 month ago

Thanks bS.

After copying beta to the latest 0.9.5 installation you must refesh your website so that it can regenerate the cache. You’ll notice that two new files where created: sitePlugins.cache.php and siteSnippets.cache.php

xWisdom
www.xwisdomhtml.com
The fear of the Lord is the beginning of wisdom:
MODx Co-Founder - Create and do more with less.

398 Posts

bS Reply #96, 20 years, 1 month ago

Warning: DocumentParser::include(D:\Tools\xampp\htdocs\modx\manager\includes/extenders/message.quit.inc.php) [function.DocumentParser-include]: failed to open stream: No such file or directory in D:\Tools\xampp\htdocs\modx\manager\includes\document.parser.class.inc.php on line 2051

Warning: DocumentParser::include() [function.include]: Failed opening 'D:\Tools\xampp\htdocs\modx\manager\includes/extenders/message.quit.inc.php' for inclusion (include_path='.;D:\Tools\xampp\php\pear\') in D:\Tools\xampp\htdocs\modx\manager\includes\document.parser.class.inc.php on line 2051

if I remove the line with the false include the screen just stays blank.

PHP Version 5.1.4
MODx version 0.9.5
Version codename rev 1197

Armand Pondman
MODx Coding Team
:: Jot :: PHx

1,732 Posts

xwisdom Reply #97, 20 years, 1 month ago

hmmm,

My bad. sad

I forgot to include that file inside the uploads. Please see the attached file. If any any then please let me know.

MAny thanks

xWisdom
www.xwisdomhtml.com
The fear of the Lord is the beginning of wisdom:
MODx Co-Founder - Create and do more with less.

398 Posts

bS Reply #98, 20 years, 1 month ago

oke the message.quit.inc fixes the "error display". now i get this:

MODx encountered the following error while attempting to parse the requested resource:
« `8 ` is not numeric and may not be passed to makeUrl() »
 
Parser timing
  MySQL: 	0.0139 s s	(14 Requests)
  PHP: 	0.0971 s s	 
  Total: 	0.1110 s s

I tracked this back to the following snippet call:

[!AjaxSearch? &AS_landing=8 &showMoreResults=1!]

this is a value in the MODxHost Template currently installed by default from the latest SVN trunk (as posted a couple of posts earlier).

I changed snippet call to:

[!AjaxSearch? &AS_landing=`8` &showMoreResults=`1`!]

added the backtick (`) and now it’s working fine. I guess if I don’t use backtick the parser takes "8 " (number eight and a space) as the value. My guess is this is a mistake in ajaxsearch snippet (it needs to Trim() the value for AS_Landing before it processes.

The first error message (see top of this post) is a bit ambigious because it only gives an error message but it doesn’t specify where.
Unless I set debug to true wink

Is this normal behaviour?

Armand Pondman
MODx Coding Team
:: Jot :: PHx

1,732 Posts

xwisdom Reply #99, 20 years, 1 month ago

Many thanks bS,

I’ll have those errors corrected in beta 3.

In your first post (about missing file) the system did show the location of the error:

"document.parser.class.inc.php on line 2051"

Thanks for the feedback

xWisdom
www.xwisdomhtml.com
The fear of the Lord is the beginning of wisdom:
MODx Co-Founder - Create and do more with less.

398 Posts

bS Reply #100, 20 years, 1 month ago

Quote from: xwisdom at Aug 07, 2006, 09:45 AM

In your first post (about missing file) the system did show the location of the error:

"document.parser.class.inc.php on line 2051"

yes but I ment the "parsing error" about the ".. `8 ` is not numeric and may not be passed to makeUrl() .. " message I was receiving.
There was no reference on screen of where this occured. When using debug=true in the index.php I could track it to the ajax-search snippet.
Maybe it’s better to just throw a "microsoft" error like "An error occured" unless debug = true tongue

Armand Pondman
MODx Coding Team
:: Jot :: PHx