FLUID-2343: that.locate(); appears to return the container if you pass it a key with a null value

Metadata

Source
FLUID-2343
Type
Bug
Priority
Major
Status
Closed
Resolution
Incomplete
Assignee
Antranig Basman
Reporter
Eli Cochran
Created
2009-03-12T20:36:00.000-0400
Updated
2011-02-22T16:27:50.109-0500
Versions
  1. 0.8
Fixed Versions
  1. 1.0
Component
  1. Framework

Description

In the Uploader, I attempted to attach an event to an optional element (in this case the Done button) using that.locate("doneButton") with the key doneButton assigned to null.

I expected to get an error or an empty jQuery but instead got the container handed back to me.

Comments

  • Antranig Basman commented 2009-03-17T17:39:18.000-0400

    I added a test case at revision 6769 to verify the current DOM binder functionality in this case, but it appears to be correct - the correct way to express dependence on a possibly missing markup element, is to configure either the selector, or the markup which is attached to a particular key, not to erase the name of the key itself - a component developer should not become aware of whether a piece of markup is configured or not except through the return from the DOM binder. Calling the DOM binder with a null value as a key is not part of its API.

  • Michelle D'Souza commented 2011-02-22T16:27:50.107-0500

    Closing issues that were resolved for 1.0 and earlier releases.