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

[ticket/16169] Add support for record filtering in text reparser #5697

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JoshyPHP
Copy link
Contributor

@JoshyPHP JoshyPHP commented Sep 27, 2019

This is against master but the API is backward compatible. I lied, I don't think it is because of the manager.

PHPBB3-16169

Checklist:

  • Correct branch: master for new features; 3.3.x & 3.2.x for fixes
  • Tests pass
  • Code follows coding guidelines: master, 3.3.x and 3.2.x
  • Commit follows commit message format

Tracker ticket (set the ticket ID to your ticket ID):

https://tracker.phpbb.com/browse/PHPBB3-16169

@JoshyPHP
Copy link
Contributor Author

JoshyPHP commented Sep 27, 2019

The code is actually old; I haven't touched it in a while. After re-reading it, I think it's incomplete because the reparser manager looks like it needs to be updated

@JoshyPHP JoshyPHP force-pushed the ticket/16169 branch 2 times, most recently from 2e1919d to faad86e Compare September 28, 2019 18:10
@rxu
Copy link
Contributor

rxu commented Nov 22, 2020

@JoshyPHP any plans on this?

@JoshyPHP
Copy link
Contributor Author

@rxu Not in the foreseeable future, no. As I recall, the filtering stuff is done but there's something about the way resume data is saved that requires some change. I can't recall the specifics but I found that the manager didn't work the way I thought and I think it required more work than anticipated or maybe the way the data is saved in the database had to be changed which meant it needed a migration and/or some backward-incompatible change and I couldn't see it through.

@marc1706
Copy link
Member

Should we close this then?

@JoshyPHP
Copy link
Contributor Author

Keep it open for now. I rebased it on current master (before that, the test suite failed due to an unrelated error) and I'll take a second pass sometime in the foreseeable future.

@JoshyPHP JoshyPHP force-pushed the ticket/16169 branch 7 times, most recently from 8785204 to 8705eb2 Compare November 30, 2020 02:04
@JoshyPHP
Copy link
Contributor Author

JoshyPHP commented Nov 30, 2020

@marc1706 I think this is about done but it will have to be rebased/fixed after #6084 is merged.

There's a summary at https://area51.phpbb.com/phpBB/viewtopic.php?f=146&t=76626

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants