Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 386340

Summary: Deferred.all should be static
Product: [ECD] Orion Reporter: Mark Macdonald <mamacdon>
Component: ClientAssignee: Simon Kaegi <simon_kaegi>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: simon_kaegi
Version: 1.0   
Target Milestone: 1.0 M1   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Mark Macdonald CLA 2012-07-31 12:07:16 EDT
'all' should be a static method that returns a new Deferred. Currently it is an instance method of Deferreds, which leads to weird code like

> new Deferred().all(promises).then( ... )

This will also align more closely with dojo/promise/all [1], Q.all [2], etc.

[1] http://livedocs.dojotoolkit.org/dojo/promise/all
[2] https://github.com/kriskowal/q/#combination
Comment 1 Simon Kaegi CLA 2012-08-01 18:57:11 EDT
Fixed in HEAD