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.