Just to be sure : it was that flaw that makes the "else" doesn't
trigger ?

On 11 avr, 23:21, Simon Vart <[email protected]>
wrote:
> Yes, I was aware of that - ArrActif shouldn't be empy in the first
> place, that's because we're adding pages that doesn't suppose to
> exists. I was trying to adapt the script without too much bothering.
>
> But your answer makes me thinks of a much better idea.
>
> I won't adapt the script at all !
>
> I will test the page displayed on and then serve the script in his
> simpliest form :http://mootools.net/shell/yYpHz/
>
> And if it's one of the added pages that shouldn't be here, I will
> serve another script, without the ArrActif.
>
> Would you think it's a better design ?
>
> On 11 avr, 23:03, Sanford Whiteman <[email protected]>
> wrote:
>
> > > I don't know if 'im doing a logical or syntaxic error in the script :
>
> > Both?
>
> > ArrActif can be an empty Array, but you call .get on its first element
> > anyway.  This  throws  a  reference  error that you should see in your
> > console.  You  have  to  make  sure  to  short-circuit before you call
> > methods that don't exist.
>
> >http://mootools.net/shell/zEAam/4/
>
> > (Changed  as  minimally  as possible to work, I don't really think you
> > have an optimal desigh here.)
>
> > -- Sandy


-- 
To unsubscribe, reply using "remove me" as the subject.

Reply via email to