Opened 14 years ago

Closed 14 years ago

#2330 closed defect (wontfix)

Error with mixed case custom widget namespace

Reported by: [email protected] Owned by: bill
Priority: high Milestone:
Component: Widgets Version:
Keywords: Cc:
Blocked By: Blocking:

Description

When creating and loading widgets from an custom mixed case namespace there always occurs the error "Error: no frag for widget type ..."

For example when loading the widget "irian.mindTracker.widget.InlineEditDialog?" using dojo.widget.createWidget("irian.mindTracker:InlineEditDialog", {...});

As it turned out dojo.widget.createWidget uses the "var ns" without applying toLowerCase() to create "var namespacedName" which ends up in a still mixed case "frag" property for the namespace. But dojo.widget.DomWidget?.getFragNodeRef() uses getNamespacedType() to check if the property exists - and getNamespacedType applies toLowerCase().

So I changed namespacedName in dojo.widget.createWidget to use a lower cased namespace and it works. Maybe this also is connected to #1726?

Attachments (1)

mixedCaseWidgetNamespace.diff (587 bytes) - added by [email protected] 14 years ago.
Path for widget.Parse

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by [email protected]

Path for widget.Parse

comment:1 Changed 14 years ago by [email protected]

Added the one-word patch which solved this for me... If usable and required for a one-word patch i also could sign a CLA...

comment:2 Changed 14 years ago by [email protected]

CLA signed...

comment:3 Changed 14 years ago by Adam Peller

Resolution: wontfix
Status: newclosed

The parser has been rewritten for Dijit. The createWidget call is gone now, and widgets are created directly with a constructor, so I don't think this applies anymore.

Note: See TracTickets for help on using tickets.