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.

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

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

Merged

Conversation

Sesquipedalian
Copy link
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
Member Author

@sbulen

@sbulen
Copy link
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
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
6 checks passed
@Sesquipedalian
Copy link
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.

@Sesquipedalian Sesquipedalian deleted the 3.0/search_topic branch February 14, 2025 22:47
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