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.

Sign up for GitHub

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

Jump to bottom

Pass current $col and $primary_keys down to the SearchReplacer #127

Closed
1 task done
drzraf opened this issue Sep 11, 2019 · 0 comments
Closed
1 task done

Pass current $col and $primary_keys down to the SearchReplacer #127

drzraf opened this issue Sep 11, 2019 · 0 comments

Comments

@drzraf
Copy link

drzraf commented Sep 11, 2019 •
edited
Loading

Feature Request

Describe your use case and the problem you are facing

When using the (to-be-merged) callback feature, I'd like to access current information about the data being replaced (post-ID, meta-ID, ...)

Describe the solution you'd like

This is possible if run_recursively / run functions of the SearchReplacer are given the information about current $col and $primary_keys
@drzraf drzraf changed the title Pass current $col and $primary_keys down the replace stack Pass current $col and $primary_keys down to the SearchReplacer Sep 11, 2019
@drzraf drzraf mentioned this issue Sep 18, 2019
Closed
drzraf added a commit to drzraf/search-replace-command that referenced this issue Jun 12, 2023
@drzraf
Added --where flag to finely select rows aimed for replacement
5c40a62 
Added --revision / --no-revision
Added --callback

Reroll wp-cli#104, wp-cli#128
Fixes wp-cli#125, wp-cli#127, wp-cli#142
@drzraf drzraf mentioned this issue Jun 12, 2023
Closed
@danielbachhuber danielbachhuber closed this as not planned Won't fix, can't repro, duplicate, stale Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@danielbachhuber @drzraf