[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [ic] parse_page: Error parsing page.


Ton Verhagen wrote:
Quoting Ton Verhagen (myself)
At 04:58 PM 2/14/2007, you wrote:
Dear All,

Creating new pages in the latest CVS version of IC
results in an "parse_page: Error parsing page" error.

This can be duplicated in de live demo.


The latest change to ContentEditor.pm can't be correct imho
as the parse_page routine returns an array of component names
(@compnames) see line 689 of ContentEditor.pm

This array could be empty of course and thus causing the test to fail...
resulting in the ever present "parse_page: Error parsing page" error.

It doesn't get that far for new pages, the piece of code where
it returns from parse_page is below:

my $body = delete $pref->{ui_body};

#::logDebug("page body=" . uneval($body));
	unless(defined $body) {
		### Already parsed, match slots and leave if not new page
		match_slots($pref, $tref);
#::logDebug("pref now=" . uneval($pref));
		return unless $opt->{new};
	}

Why $opt->{new} isn't set, no idea.




5c5
< # $Id: ContentEditor.pm,v 2.18 2006/05/04 20:07:32 mheins Exp $
---
 > # $Id: ContentEditor.pm,v 2.17 2005/11/08 18:14:44 jon Exp $
27c27
< $VERSION = substr(q$Revision: 2.18 $, 10);
---
 > $VERSION = substr(q$Revision: 2.17 $, 10);
650,653d649
<       if($scratches->{not_editable} and $vals->{not_editable}) {
<               return death('controls', "Not editable page");
<       }
<
2879,2886c2875
<
<       ## If returns false then must be error or not editable
<       parse_page($pref, $opt)
<               or do {
< Vend::Tags->error({ name => 'parse_page', set => "Error parsing page." });
<                       return;
<               };
<
---
 >       parse_page($pref, $opt);


Please use context diffs (diff -u).

Bye
	Racke


--
LinuXia Systems => http://www.linuxia.de/
Expert Interchange Consulting and System Administration
ICDEVGROUP => http://www.icdevgroup.org/
Interchange Development Team

_______________________________________________
interchange-users mailing list
suppressed
http://www.icdevgroup.org/mailman/listinfo/interchange-users


Mail converted by mhonarc 2.6.15
This archive provided courtesy of JSW4.NET, Internet Hosting Services for Small Business.