[ic] flypage specialsub patch
Gert van der Spoel
gert at 3edge.com
Tue Sep 1 06:49:37 UTC 2009
> -----Original Message-----
> From: interchange-users-bounces at icdevgroup.org [mailto:interchange-
> users-bounces at icdevgroup.org] On Behalf Of Peter
> Sent: Monday, August 31, 2009 11:06 PM
> To: interchange-users at icdevgroup.org
> Subject: Re: [ic] flypage specialsub patch
> On 08/31/2009 12:01 PM, Mike Heins wrote:
> > Quoting Jon Jensen (jon at endpoint.com):
> >> On Mon, 31 Aug 2009, Mike Heins wrote:
> >>> Looks like a good fix to me. I can't imagine that it would cause a
> >>> problem even for someone currently using the SpecialSub.
> >> It looks more reasonable to me than the old code, but the behavior
> >> quite look compatible. Before, $base could be assigned an arbitrary
> >> hashref and thus could be used for a different kind of on-the-fly
> >> but now it seems the product code has to actually exist in a real
> >> database.
> > Good point.
> >> I am not using that feature and don't object to the change, but I
> >> I'd point out the different behavior.
> > Now that I think about it, it was designed to handle OnFly products
> so the
> > change doesn't make sense.
> Ok, well that's why I asked. How about we keep the old code if the
> return is a hashref, but if it's just a text scalar which contains a
> then get $base from product_code_exists_ref()? Also just to give an
> explicit way to say, "don't display a flypage for this" a return of
> undef could result in the flypage being skipped as if the sku doesn't
More information about the interchange-users