We need to port the Find Replace Regex Plugin from D7 to D8.

CommentFileSizeAuthor
#4 find_replace_regex-2975179-4.patch7.55 KBvijay.mayilsamy
#2 plugin-page-1 .png73.98 KBsivaramakrishnan
#2 plugin-page-2.png55.3 KBsivaramakrishnan
#2 plugin-page-3.png53.19 KBsivaramakrishnan
#2 2975179_1.patch3.4 KBsivaramakrishnan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jamesdixon created an issue. See original summary.

sivaramakrishnan’s picture

sivaramakrishnan CreditAttribution: sivaramakrishnan as a volunteer and commented
Status: Active » Needs review
FileSize
2975179_1.patch3.4 KB
plugin-page-3.png53.19 KB
plugin-page-2.png55.3 KB
plugin-page-1 .png73.98 KB

Please Apply the Here attached Patch and this patch for port the Find Replace Regex Plugin from D7 to D8.

ericgsmith’s picture

ericgsmith CreditAttribution: ericgsmith as a volunteer commented
Status: Needs review » Needs work

Hi @sivaramakrishnan

Thanks for the patch! This looks like a good start.

Each plugin needs a unit test (see the tests/src/Unit/Plugin/Tamper folder for examples).

This will allow us to validate the plugin is working. I'll take a closer look at the code once we have a passing test - please let me know if you are willing and able to add one.

vijay.mayilsamy’s picture

vijay.mayilsamy CreditAttribution: vijay.mayilsamy as a volunteer and at Dialed In Design commented
FileSize
find_replace_regex-2975179-4.patch7.55 KB

Hi @ericgsmith,

Here is the patch for Find Replace Regex Plugin to D8.

Thanks
Vijay

vijay.mayilsamy’s picture

vijay.mayilsamy CreditAttribution: vijay.mayilsamy as a volunteer and at Dialed In Design commented
Status: Needs work » Needs review
ericgsmith’s picture

ericgsmith CreditAttribution: ericgsmith as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Thanks all :)

ericgsmith’s picture

ericgsmith CreditAttribution: ericgsmith as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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