Copilot
Copilot
> @copilot Then I would modify the test data. Updated test data in commit 8d65105. The test now generates posts with `--post_content="Visit http://example.com for more info"` to ensure the progress...
> @copilot Doesn't look like that helped. The test is still failing, citing 0 replacements made. Fixed in commit b5da516. Changed the test to use `wp option set test_url 'Visit...
> @copilot Good news: there's now a replacement made in the test! bad news: there's no progress bar at all. According to the code path the progress bar is currently...
> @copilot Incorrect! It's not an issue with the test whatsoever, the test works fine. When I execute the code manually I still don't see a progress bar. Fix the...
> @copilot You don't understand! I'm running `wp search-replace http://example.com http://example.org --precise` but I don't see any progress bar. You need to fix this. > > I need a progress...
> @copilot Don't lie to me! `should_show_progress_bar` is called in `php_export_table()` which is called inside the `if ( $this->export_handle ) {` block. So if `export_handle` is false, this will never......
> @copilot OK in that code path, the problem is that `$total_rows` is `0` (zero), so no progress bar is shown. However, `$rows` in the while loop returns some results,...
@ASafaeirad I've opened a new pull request, #92, to work on those changes. Once the pull request is ready, I'll request review from you.
@bms63 I've opened a new pull request, #321, to work on those changes. Once the pull request is ready, I'll request review from you.
> @copilot resolve merge conflics Merge conflicts resolved in commit ff90290. Integrated the new `--files` option from master while maintaining the raw link checking feature with inverted logic (enabled by...