Bad return value of Event.Behavior match()

#68

This issues append since a few month, I can't gives the version number because LivePipe do not have release version number.

Sample:

  with(Event.Behavior){  
    hide('db_configuration').when('db_jcmsdb').is('derby');  
  }

This code always return "false" because of function match():

  return this.conditions.inject(false, function (bool,condition) {
    return (condition[0] == 'and') ? (bool && condition[1]()) : (bool || condition[1]());
  });

Previous version was initialise with "new Boolean()":

  return this.conditions.inject(new Boolean(), function (bool,condition) {
    return (condition[0] == 'and') ? (bool && condition[1]()) : (bool || condition[1]());
  });

The default condition is 'and' (initialised l30) so: false && anything = false. In previous version, new Boolean() == true

Questions:
- Is there official version number ? - Is there official release ? and notification of these releases ?

Reported by Jean-Philippe Encausse · September 25th, 2009 @ 05:21 AM

Activity

Nathan L Smith
- Assigned user changed from Ryan Johnson to Nathan L Smith
- State changed from new to resolved
Jean-Phillippe:

This should be fixed in 1591c1e. Please reopen the ticket if it still doesn't work.
- Is there official version number ?
No. The release linked to on the site is that of the most recent master branch on github (http://github.com/syntacticx/livepipe-ui)
- Is there official release ?
No, but I'm working to get the existing bugs fixed and verification of IE 8 and Prototype 1.6.1 compatibility before tagging a 1.0 release. After that there might be a more formal release schedule. I'd also like to set up a test suite to be able to catch problems like this (I think I broke this one while JSLinting the file and not thinking (sorry.))

and notification of these releases ?

If you watch the project on github you'll be able to see any changes in the feed.

Thanks for the fix. Any other help would be appreciated. I've sort of taken over on maintenance of the project but don't have very much time, so if there are others willing to follow the google groups and tickets it would be great.
October 22nd, 2009 @ 10:49 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.