David Perez
Forum Replies Created
-
Hello,
I’ll check it again and I’ll reply you accordingly.
Thanks!
Ok! thanks
Hello,
It does not work. That’s why I added this ticket. Even if you select the option default, it loads at frontend the font Nunito. Could you review with your team?
Si justo Albert, pero puedes probarlo tú mismo… 😉
Buenas Albert,
Te dejo un pequeño video para que veas cómo funciona la configuración:
https://close.technology/documentacion/formscrm/
Un saludo,
Forum: Reviews
In reply to: [Plugin Check (PCP)] False positivesHello,
You’re rating a plugin that you feel is not accurate enough. However, this is a community plugin. If you feel that something is missing or a check is inaccurate, please add an issue here with all the relevant details.
https://github.com/WordPress/plugin-check/issues.
You could even add a PR of the improvement! It’s a tool from developers for developers!
We have moved this check to a warning status, as we are working on improving the AI to reduce the number of false positives. Until this enhancement is implemented, we don’t envisage any other solutions.
Hello Tijmen,
You should always use the plugin slug as the text domain. This will not affect old translations, but you will see them translated in your plugin.
Forum: Plugins
In reply to: [Plugin Check (PCP)] Use placeholders and $wpdb->prepare(); found $queryHello,
The issue is that Plugin Check (PCP) requires prepared statements to be passed directly to the database method – it doesn’t recognize them when stored in a variable first.
The correct solution:
Pass the prepared statement directly to
get_results():$table = $wpdb->prefix . '_visitors'; $t4 = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM %i WHERE date = %s ORDER BY n DESC", $table, $custom_day ) );This is the only way PCP will accept it. You cannot:
- Store the query in a variable first
- Use ignore comments (PCP doesn’t allow them)
Additional notes:
- Make sure you’re using
%ifor identifiers (table/column names) – requires WordPress 6.2+ - Use
%s,%d, or%ffor values - Column names like
dateandnare correctly hardcoded in your query
This pattern ensures PCP can verify your code is properly using prepared statements.
Regards
- This reply was modified 3 months, 3 weeks ago by David Perez.
Forum: Plugins
In reply to: [Plugin Check (PCP)] Non-global variables seen as global variablesHello Jose,
I don’t think so. It’s as we said.
Forum: Plugins
In reply to: [Plugin Check (PCP)] Non-global variables seen as global variablesHello Jose,
We are actively developing PCP, but it is not always easy to refine the checks. That’s why we’ve moved it to a warning for now, while we look for a better solution. It won’t be a quick solution.
This plugin is a community project, so if you think you can improve it, please do!
Here is the repository: https://github.com/WordPress/plugin-check.
Forum: Plugins
In reply to: [Plugin Check (PCP)] missing_direct_file_access_protection needs workThis is the first version. We will improve it based on the cases you mentioned. Stay tuned!
Forum: Plugins
In reply to: [Plugin Check (PCP)] missing_direct_file_access_protection needs workHello,
They are not false positive as we are asking authors to ensure that their php file is not called directly.
It’s nice to hear that you managed to solve it.
Where is that function? is it global? You need to share all context.