getAllChildren bug on space CSS selector

[c2c]

As I read on this groups, using a CSS selector like "#nonexistantDiv
span" where #nonexistantDIv points to nothing, getAllChildren throws
an error about "e" not being defined.

As just returning an empty array like this :

function getAllChildren(e) {
  // Returns all children of element. Workaround required for IE5/
Windows. Ugh.
  if (!e) {
  return new Array();
  }
  return e.all ? e.all : e.getElementsByTagName('*');
}

gives a solution, wouldn't it be nice to put it in behaviour.js ?

Regards,

--
Julien CROUZET aka c2c
julien/at\theoconcept.com


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Behaviour Javascript Library" group.
To post to this group, send email to behaviour@...
To unsubscribe from this group, send email to behaviour-unsubscribe@...
For more options, visit this group at http://groups.google.com/group/behaviour?hl=en
-~----------~----~----~----~------~----~------~--~---

It certainly would. :) I'll see what I can do. :)

Ben

On 8/18/07, c2c <julien.crouzet@...> wrote:

As I read on this groups, using a CSS selector like "#nonexistantDiv
span" where #nonexistantDIv points to nothing, getAllChildren throws
an error about "e" not being defined.

As just returning an empty array like this :

function getAllChildren(e) {
  // Returns all children of element. Workaround required for IE5/
Windows. Ugh.
  if (!e) {
        return new Array();
  }
  return e.all ? e.all : e.getElementsByTagName('*');
}

gives a solution, wouldn't it be nice to put it in behaviour.js ?

Regards,

--
Julien CROUZET aka c2c
julien/at\theoconcept.com

--
Regards,
Ben Nolan

http://bennolan.com/

skype: bennolan
cell: [phone]
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Behaviour Javascript Library" group.
To post to this group, send email to behaviour@...
To unsubscribe from this group, send email to behaviour-unsubscribe@...
For more options, visit this group at http://groups.google.com/group/behaviour?hl=en
-~----------~----~----~----~------~----~------~--~---

[c2c]

On 18 août, 21:11, "Ben Nolan" <bno...@...> wrote:
> It certainly would. :) I'll see what I can do. :)
>

Thanks, adding thins manually on each Behaviour update is
not the best solution ... :)

Thanks again,

--
Julien CROUZET aka c2c
julien/at\theoconcept.com


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Behaviour Javascript Library" group.
To post to this group, send email to behaviour@...
To unsubscribe from this group, send email to behaviour-unsubscribe@...
For more options, visit this group at http://groups.google.com/group/behaviour?hl=en
-~----------~----~----~----~------~----~------~--~---