Re: [ic] parse_page: Error parsing page.

[Ton Verhagen <suppressed>]

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    suppressed    http://www.alamerce.nl

eCommerce Solutions -- Consulting -- Website & Interchange Hosting 


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