Opened 10 years ago
Last modified 6 years ago
#16235 assigned enhancement
[patch] [cla] dojox/geo/openlayers AMD
Reported by: | Philippe May | Owned by: | dylan |
---|---|---|---|
Priority: | low | Milestone: | 1.15 |
Component: | Dojox | Version: | 1.8.1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
The dojox/geo/openlayers currently requires OpenLayers? to be loaded through a <script> tag, breaking the principles of dynamic module loading.
Incidentally there is an option to build OpenLayers? for AMD compatibility in the recent evolutions (development version, to be included in the next release).
Dojox/geo/openlayers should be accepting the 2 options.
Proposed patches attached to this ticket, they consist of:
- a loader plugin (maybeamd.js) which loads the AMD OL version, or just returns the globally defined one
- minor modifications of the existing files, defining "OpenLayers?" as a local variable resulting of the plugin
- README file describing the process involved, specifically a "openlayers-amd" configuration variable, set through dojoConfig or "has". If the "openlayers-amd" option is not set, OpenLayers? is retrieved form the global environment, a legacy fallback.
Attachments (5)
Change History (14)
Changed 10 years ago by
Attachment: | README.rst added |
---|
Changed 10 years ago by
Attachment: | maybeamd.js added |
---|
Changed 10 years ago by
Attachment: | openlayer-amd-patch.2.diff added |
---|
Changed 10 years ago by
Attachment: | openlayer-amd-patch.diff added |
---|
Please consider this patch file (debugger statement removed)
comment:1 Changed 10 years ago by
Owner: | changed from Adam Peller to Tom Trenka |
---|---|
Status: | new → assigned |
comment:2 Changed 10 years ago by
Aprilmay, looks like you haven't filed a CLA, can you do that first?
comment:3 Changed 10 years ago by
I filled the CLA. I'm also adding a quick test for the proposed evolution.
Changed 10 years ago by
Attachment: | test_amd.html added |
---|
comment:4 Changed 10 years ago by
Summary: | dojox/geo/openlayers AMD → [patch] [cla] dojox/geo/openlayers AMD |
---|
comment:5 Changed 9 years ago by
Owner: | changed from Tom Trenka to cjolif |
---|
comment:6 Changed 8 years ago by
Owner: | cjolif deleted |
---|---|
Status: | assigned → open |
I definitely don't have/won't have the time to look into this one.
comment:7 Changed 8 years ago by
Milestone: | tbd → 1.11 |
---|---|
Owner: | set to dylan |
Priority: | undecided → low |
Status: | open → assigned |
comment:8 Changed 7 years ago by
Milestone: | 1.11 → 1.12 |
---|
comment:9 Changed 6 years ago by
Milestone: | 1.13 → 1.15 |
---|
Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.
Please consider this patch file (debugger statement removed)