#68 ✓resolved
Jean-Philippe Encausse

Bad return value of Event.Behavior match()

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

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 ?

Comments and changes to this ticket

  • Nathan L Smith

    Nathan L Smith October 22nd, 2009 @ 10:49 AM

    • 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.

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.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

NOTE: Please do not open tickets here. If you have an issue, please open it on http://github.com/syntacticx/livepipe-ui/issues/

High quality widgets and controls for web 2.0 applications built using the Prototype JavaScript Framework. Each control is well tested, highly extensible, fully documented and degrades gracefully for non JavaScript enabled browsers where possible. MIT licensed and actively maintained.

People watching this ticket

Pages