Closed
Bug 1119842
Opened 9 years ago
Closed 9 years ago
Refactor fxpay module exports
Categories
(Marketplace Graveyard :: Payments/Refunds, defect, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: kumar, Unassigned)
References
Details
(Whiteboard: [qa-])
fxpay has pseudo modules that do this: exports.foo = {} exports.someFunction = function() { ... } Let's refactor that so we export everything in one place, more like: exports.foo = { someFunction: someFunction, otherFunction: otherFunction, } function someFunction() { } ... This makes modules more self-contained and explicit. Instead of calling exports.helper() internally they can just call helper(). It will also put all public exports in one place which is easier when you're looking for symbols. It also encourages the use of named functions which helps in certain debugging scenarios.
Updated•9 years ago
|
Priority: -- → P3
Reporter | ||
Comment 1•9 years ago
|
||
We learned in bug 1137584 that we kind of need to use exports because of circular dependencies. Oh well.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•