Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add in delete:entry command #4

Merged
merged 4 commits into from
Oct 9, 2014
Merged

Conversation

khaliqgant
Copy link
Contributor

Adds in the delete:entry command by using the channel_entries api. Super admin status must be set and also the can_delete_all_entries must be set to yes as well

        ee()->session->userdata['group_id'] = '1';
        ee()->session->userdata['can_delete_all_entries'] = 'y';

The next step to this which I'll add in later is the ability to pass in multiple entries to delete.

@rsanchez
Copy link
Owner

rsanchez commented Oct 5, 2014

This is great. I think we should get rid of the delete by title option and just leave it as entry_id or url_title.

@khaliqgant
Copy link
Contributor Author

Cool, sounds good to me! I actually forget to mention the bug I noticed in testing that upon deleting, the confirmation that is successful does not fire https://github.com/khaliqgant/eecli/blob/develop/src/Command/DeleteEntryCommand.php#L74

Something I'll look into further.

@khaliqgant
Copy link
Contributor Author

Ability to delete by title has been removed. Only able to delete by entry_id and url_titlle

rsanchez added a commit that referenced this pull request Oct 9, 2014
Add in delete:entry command
@rsanchez rsanchez merged commit 70519dc into rsanchez:develop Oct 9, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants