diff options
| author | Xemle <xemle@phtagr.org> | 2013-05-27 17:01:48 +0200 |
|---|---|---|
| committer | Xemle <xemle@phtagr.org> | 2013-05-27 17:01:48 +0200 |
| commit | ce88338925138eb0fc1ad9f68ca7d55b42f3a884 (patch) | |
| tree | 8c22fd844b8c10684be22d7b0a9e913085436594 /js | |
| parent | 6ede787f3a0778f5542573e93e881894ec67aad5 (diff) | |
Issue #166 Improve RESTful urls for web backend
Change create and delete urls for feeds and folders:
POST /feed - Creates a feed
DELETE /feed/:feedId - Deletes a feed
POST /folder - Creates a folder
DELETE /folder/:folderId - Deletes a feed
Diffstat (limited to 'js')
| -rw-r--r-- | js/app/services/persistence.coffee | 4 | ||||
| -rw-r--r-- | js/public/app.js | 4 | ||||
| -rw-r--r-- | js/tests/services/persistenceSpec.coffee | 5 |
3 files changed, 7 insertions, 6 deletions
diff --git a/js/app/services/persistence.coffee b/js/app/services/persistence.coffee index 98549b98f..9d94daea8 100644 --- a/js/app/services/persistence.coffee +++ b/js/app/services/persistence.coffee @@ -197,7 +197,7 @@ $rootScope) -> routeParams: feedId: feedId - @_request.post 'news_feeds_delete', params + @_request.delete 'news_feeds_delete', params restoreFeed: (feedId, onSuccess=null) -> @@ -329,7 +329,7 @@ $rootScope) -> routeParams: folderId: folderId - @_request.post 'news_folders_delete', params + @_request.delete 'news_folders_delete', params restoreFolder: (folderId, onSuccess=null) -> diff --git a/js/public/app.js b/js/public/app.js index 1d597f2ad..411814bcc 100644 --- a/js/public/app.js +++ b/js/public/app.js @@ -2668,7 +2668,7 @@ License along with this library. If not, see <http://www.gnu.org/licenses/>. feedId: feedId } }; - return this._request.post('news_feeds_delete', params); + return this._request["delete"]('news_feeds_delete', params); }; Persistence.prototype.restoreFeed = function(feedId, onSuccess) { @@ -2852,7 +2852,7 @@ License along with this library. If not, see <http://www.gnu.org/licenses/>. folderId: folderId } }; - return this._request.post('news_folders_delete', params); + return this._request["delete"]('news_folders_delete', params); }; Persistence.prototype.restoreFolder = function(folderId, onSuccess) { diff --git a/js/tests/services/persistenceSpec.coffee b/js/tests/services/persistenceSpec.coffee index 2f1607cff..0d0880a2b 100644 --- a/js/tests/services/persistenceSpec.coffee +++ b/js/tests/services/persistenceSpec.coffee @@ -28,6 +28,7 @@ describe 'Persistence', -> beforeEach module ($provide) => @req = get: jasmine.createSpy('get') + delete: jasmine.createSpy('delete') post: jasmine.createSpy('post') @config = itemBatchSize: 3 @@ -204,7 +205,7 @@ describe 'Persistence', -> @Persistence.deleteFeed(params.routeParams.feedId) - expect(@req.post).toHaveBeenCalledWith('news_feeds_delete', params) + expect(@req.delete).toHaveBeenCalledWith('news_feeds_delete', params) it 'send a correct feed restore request', => @@ -302,7 +303,7 @@ describe 'Persistence', -> @Persistence.deleteFolder(params.routeParams.folderId) - expect(@req.post).toHaveBeenCalledWith('news_folders_delete', params) + expect(@req.delete).toHaveBeenCalledWith('news_folders_delete', params) it 'send a correct folder restore request', => |