Opened 13 years ago

Closed 12 years ago

#2330 closed defect (wontfix)

Error with mixed case custom widget namespace

Reported by: st.schuster@… 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 st.schuster@… 13 years ago.
Path for widget.Parse

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by st.schuster@…

Path for widget.Parse

comment:1 Changed 13 years ago by st.schuster@…

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 12 years ago by st.schuster@…

CLA signed...

comment:3 Changed 12 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.