Opened 10 years ago

Closed 10 years ago

#14882 closed defect (fixed)

dojo.cldr.monetary unnecessarily includes Dojo base [patch]

Reported by: Stefan Bird Owned by: bill
Priority: undecided Milestone: 1.8
Component: General Version: 1.7.2
Keywords: Cc:
Blocked By: Blocking:

Description

On dojo/cldr/monetary.js:1 it imports dojo/main, when it only needs dojo/_base/lang. This means that Nano builds that use monetary will pull in dojo/main.js, negating the advantages of a Nano build.

Attachments (1)

monetary.js.patch​ (658 bytes) - added by Stefan Bird 10 years ago.

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by Stefan Bird

Attachment: monetary.js.patch​ added

comment:1 Changed 10 years ago by bill

Milestone: tbd → 1.8

The patch is a bit out of date but I'll check in the essence of it, thanks for catching the problem.

Let us know if you see any other problems, although if you have any significant patches (more than a few lines) please file a ​cla first.

comment:2 Changed 10 years ago by bill

Owner: set to bill
Resolution: → fixed
Status: new → closed

In [28003]:

granular dependencies, fixes #14882.

Note: See TracTickets for help on using tickets.