Opened 14 years ago
Closed 14 years ago
#2330 closed defect (wontfix)
Error with mixed case custom widget namespace
Reported by: | 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)
Change History (4)
Changed 14 years ago by
Attachment: | mixedCaseWidgetNamespace.diff added |
---|
comment:1 Changed 14 years ago by
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:3 Changed 14 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
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.
Path for widget.Parse