Apply pipeline to db api endpoint #5972

delgermurun · 2015-10-20T12:25:10Z

adds pipeline to export, import and delete all methods

ErisDS · 2015-10-20T18:01:09Z

I think that it should be possible to further refactor this to use utils.handlePermissions(docName, methodName) rather than having your own version? See Posts/Tags/Users for an example of what I mean

refs #5508 - adds pipeline to export, import and delete all methods

delgermurun · 2015-10-21T01:35:18Z

@ErisDS updated. Thanks for the guide.

Apply pipeline to db api endpoint

core/server/api/db.js

+        function validate(options) {
+            // Check if a file was provided
+            if (!utils.checkFileExists(options, 'importfile')) {
+                return Promise.reject(new errors.NoPermissionError('Please select a file to import.'));


Apply pipeline to db api endpoint

b37c0f2

refs #5508 - adds pipeline to export, import and delete all methods

ErisDS added a commit that referenced this pull request Oct 21, 2015

Merge pull request #5972 from delgermurun/pipeline-db

ef57d81

Apply pipeline to db api endpoint

ErisDS merged commit ef57d81 into TryGhost:master Oct 21, 2015

delgermurun deleted the pipeline-db branch October 21, 2015 21:47

delgermurun reviewed Oct 21, 2015
View reviewed changes

ErisDS mentioned this pull request Nov 3, 2015

Regression: Export throws 'access denied' #6040

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Apply pipeline to db api endpoint #5972

Apply pipeline to db api endpoint #5972

delgermurun commented Oct 20, 2015

ErisDS commented Oct 20, 2015

delgermurun commented Oct 21, 2015

This comment was marked as abuse.

This comment was marked as abuse.

Apply pipeline to db api endpoint #5972

Apply pipeline to db api endpoint #5972

Conversation

delgermurun commented Oct 20, 2015

ErisDS commented Oct 20, 2015

delgermurun commented Oct 21, 2015

This comment was marked as abuse.

This comment was marked as abuse.