Skip to content

[3.0] Returns correct search results when post moderation is enabled#8459

Merged
Sesquipedalian merged 1 commit intoSimpleMachines:release-3.0from
Sesquipedalian:3.0/search_topic
Feb 14, 2025
Merged

[3.0] Returns correct search results when post moderation is enabled#8459
Sesquipedalian merged 1 commit intoSimpleMachines:release-3.0from
Sesquipedalian:3.0/search_topic

Conversation

@Sesquipedalian
Copy link
Copy Markdown
Member

Fixes #7870 for SMF 3.0.

Closes #8333.

Signed-off-by: Jon Stovell <jonstovell@gmail.com>
@Sesquipedalian Sesquipedalian added this to the 3.0 Alpha 3 milestone Feb 11, 2025
@Sesquipedalian Sesquipedalian changed the title Returns correct search results when post moderation is enabled [3.0] Returns correct search results when post moderation is enabled Feb 11, 2025
@Sesquipedalian
Copy link
Copy Markdown
Member Author

@sbulen

@sbulen
Copy link
Copy Markdown
Contributor

sbulen commented Feb 12, 2025

Note there is a deprecation warning:
Deprecated
: str_ireplace(): Passing null to parameter 3 ($subject) of type array|string is deprecated in
D:\wamp64\www\84search30\upgrade.php
on line
959

Same behavior under PHP 8.3 & 8.4.

I don't know if that is the cause, but I cannot run a 3.0 => 3.0 upgrade to test this with my existing 3.0 test env/data. I get stuck on this screen:
(And yes, I did try clicking on the "try again" link!)
image

@sbulen
Copy link
Copy Markdown
Contributor

sbulen commented Feb 12, 2025

Note that if I manually update the index, I get the same results as in #8458 , which match the current results.

That may be good enough for now, as we have at least two major upgrader.php PRs coming up, the mb4 one and the rewrite. Not sure we want to invest a lot of time in this incarnation of the upgrader. The logic appears to work.

But prior to those two PRs, looks like we can't do a 3.0 => 3.0 upgrade at the moment. Not a major loss, but, it's a test result...

@Sesquipedalian Sesquipedalian merged commit 999f3b4 into SimpleMachines:release-3.0 Feb 14, 2025
@Sesquipedalian
Copy link
Copy Markdown
Member Author

Note that if I manually update the index, I get the same results as in #8458 , which match the current results.

That may be good enough for now, as we have at least two major upgrader.php PRs coming up, the mb4 one and the rewrite. Not sure we want to invest a lot of time in this incarnation of the upgrader. The logic appears to work.

Excellent.

We'll deal with upgrader stuff elsewhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Search this topic does not deliver any results

2 participants