search-regex icon indicating copy to clipboard operation
search-regex copied to clipboard

Published 20 hours ago • verified publisher icon johngodley

Show the Post ID for the Post Meta results

Open cliffordp opened this issue 1 year ago • 10 comments

First time using the plugin, tyvm for it.

Please provide the Post information (Post ID with edit/view links) even for the Post Meta results: image

P.S. 2) Shouldn't the Row# be without a comma, in case I want to copy it? 3) Please make it more obvious that clicking on the result will perform the replacement, not just if I hover over it for a while to view the <title>. It'd be nice if there was a Read Only view/toggle so I don't do accidental clicks.

cliffordp avatar Jan 05 '24 21:01 cliffordp

Clicking on the result doesn't immediately perform the replacement.

johngodley avatar Jan 06 '24 08:01 johngodley

meaning it does eventually?

cliffordp avatar Jan 06 '24 16:01 cliffordp

Yes. Have you pressed it?

johngodley avatar Jan 06 '24 17:01 johngodley

No, just was looking for Replace All. FYI: it was not obvious it was only going to replace the page size (25 quantity) and not actually all

cliffordp avatar Jan 06 '24 17:01 cliffordp

It does replace all and not just the page size. Are we still talking about the same thing?

johngodley avatar Jan 06 '24 17:01 johngodley

OK, I tried the single click on orange and that's nice! maybe the

should change from "Click to replace match" to "Click to replace manually" <p>I found it odd, though, that clicking on it does not prompt me with the global replacement (my shortcode in this example) but instead reverts it back to the original: <img width="479" alt="image" src="https://github.com/johngodley/search-regex/assets/1812179/b5713c5a-b80e-472c-82dd-130fbbef2cb0"></p> <p>And when I tried replacing it solo (orange) it gave an error:</p> <pre><code>Plugin: 3.0.8 WordPress: 6.4.2 (single) PHP: 7.4.33 256M 300s Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36 JavaScript: /wp-content/plugins/search-regex/search-regex.js REST API: /wp-json/ Query: ?page=search-regex.php&preset=6599900cf12d8 Error: No matching row (rest_invalid_param) Raw: {"code":"rest_invalid_param","message":"No matching row","data":{"status":400}} </code></pre> <p>So I refreshed the page, performed the same search, then used Replace All and that worked.</p>

cliffordp avatar Jan 06 '24 17:01 cliffordp

If you want to replace globally then you use the 'replace all' button. Details about using the plugin can be found here:

https://searchregex.com/

johngodley avatar Jan 06 '24 18:01 johngodley

Am I understanding correctly that you don't want me to continue reporting bugs and UX improvements?

FYI: here's the Replace All doing not all but also not just 25 2024-01-06 SearchRegex Replace All not quite

cliffordp avatar Jan 06 '24 18:01 cliffordp

Am I understanding correctly that you don't want me to continue reporting bugs and UX improvements?

I don't believe I've said that.

You have reported things before trying to use them, and it is hard to know what you are currently referring to.

I am struggling to make out what you are showing in the gif.

johngodley avatar Jan 06 '24 18:01 johngodley

OK, thanks for clarifying. In the gif it shows clicking Replace All when the pagination is 25. It shows 25 rows then it all of a sudden says 152 rows, even though the initial result count was 191 rows. Upon completing the first Replace All, there are 39 rows still found upon performing the same search.

For clicking the orange, why wouldn't it pre-fill my replacement value instead?

cliffordp avatar Jan 06 '24 22:01 cliffordp