[ic] parse_page: Error parsing page.

[Ton Verhagen]

Quoting Racke:
>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.


Well, I have commented out what Mike put in on May 4th 2006
(last modification of this module) and everything started working again....

So, I presume you are mistaken...


>>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).

OK, will do next time...



Best regards,

Ton

--
Alamerce Nederland --- Cornerstone of eBusiness
Phone +31-40-2374381    info at alamerce.nl    http://www.alamerce.nl

eCommerce Solutions -- Consulting -- Website & Interchange Hosting