Problem/Motivation

Migrate methods of command `nrd-delete-prior-revisions` to NodeRevisionDeleteCliService

Proposed resolution

None.

Remaining tasks

Patch to come.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

None.

CommentFileSizeAuthor
#6 node_revision_delete-drush_nrd_dpr-3078184-6-8.x.patch5.39 KBrobert ngo
#3 Screen Shot 2019-08-31 at 7.44.25 PM.png92.02 KBrobert ngo
#3 Screen Shot 2019-08-31 at 7.44.37 PM.png139.95 KBrobert ngo
#3 Screen Shot 2019-08-31 at 7.45.58 PM.png191.03 KBrobert ngo
#3 node_revision_delete-drush_nrd_dpr-3078184-3-8.x.patch5.48 KBrobert ngo

Comments

Robert Ngo created an issue. See original summary.

adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Title: Port `nrd-delete-prior-revisions` command » Port the nrd-delete-prior-revisions drush command
robert ngo’s picture

robert ngo
Location Montreal ⚜️
commented
StatusFileSize
new node_revision_delete-drush_nrd_dpr-3078184-3-8.x.patch5.48 KB
new Screen Shot 2019-08-31 at 7.45.58 PM.png191.03 KB
new Screen Shot 2019-08-31 at 7.44.37 PM.png139.95 KB
new Screen Shot 2019-08-31 at 7.44.25 PM.png92.02 KB

Output of drush nrd-dpr --help

Output of command drush nrd-dpr with valid parameters.

Validation in action.

robert ngo’s picture

robert ngo
Location Montreal ⚜️
commented
Status: Active » Needs review
robert ngo’s picture

robert ngo
Location Montreal ⚜️
commented

This patch should be applied ONLY AFTER merging #3078169: Create NodeRevisionDeleteCliService

robert ngo’s picture

robert ngo
Location Montreal ⚜️
commented
StatusFileSize
new node_revision_delete-drush_nrd_dpr-3078184-6-8.x.patch5.39 KB

Updated patch.
No need to pass callable t() to CLI service.

  • adriancid committed f2d4edc on 8.x-1.x authored by Robert Ngo 
    Issue #3078184 by Robert Ngo, adriancid: Port the nrd-delete-prior-...
adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.