The Tokens in Views Filter Criteria module allow to use tokens in views string filter criteria values.

Project link

https://www.drupal.org/project/token_views_filter

Git instructions

git clone --branch 8.x-1.x https://git.drupal.org/project/token_views_filter.git

PAReview checklist

https://pareview.sh/pareview/https-git.drupal.org-project-token_views_fi...

Comments

validoll created an issue. See original summary.

validoll’s picture

Issue summary: View changes
avpaderno’s picture

Thank you for your contribution! Reviewers will soon check the code and report here what they found out.

klausi’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -PAreview: single application approval

Looks good to me!

TokensStringFilter: do not use \Drupal in classes, use dependency injection instead. See https://www.drupal.org/docs/8/api/services-and-dependency-injection/serv...

I think we should do a full user promotion here since there is a test case and with that enough code to justify that.

validoll’s picture

> TokensStringFilter: do not use \Drupal in classes

Oh... It looks like I missed this point. Thanks. Will be fixed in future release.

avpaderno’s picture

klausi’s picture

@kiamlaluno: I removed the "PAReview: single application approval" intentionally because I think there is enough code to fully approve the user. Any objections?

avpaderno’s picture

Well, yes... I see a file with just a line of code, one with two lines of code, and a file with 23 lines of code (of which 18 lines are for defining arrays). I didn't count the lines defining the namespaces, the use statements, or the opening/closing part of functions/methods/classes.

The project doesn't contain enough code that I could say Yes, this user has a grasp of how to use Drupal APIs and write secure code. I am not saying the OP doesn't understand how to write Drupal code, but that the code is not enough to understand how much the OP knows.

klausi’s picture

I see, there is also the test file I mentioned with 70 lines extra well structured code.

Anyway, @validoll feel free to get back to us when you have your next project to get security team support :-)

avpaderno’s picture

Assigned: Unassigned » avpaderno
Status: Reviewed & tested by the community » Fixed

Thank you for your contribution! I am going to update the project.

These are some recommended readings to help with excellent maintainership:

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

I thank all the dedicated reviewers as well.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.