Skip to content

Conversation

CySSoO
Copy link
Contributor

@CySSoO CySSoO commented Oct 2, 2025

Summary

  • expose module statistics in the configuration controller and register a dedicated tab
  • render a new documentation card that highlights block, shortcode, FAQ, tab, flag, modal and game counts

Testing

  • php -l everblock.php

https://chatgpt.com/codex/tasks/task_e_68debb5deccc8322b29518a9bb6a969a

@CySSoO CySSoO merged commit 2122475 into master Oct 2, 2025
0 of 6 checks passed
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

everblock/everblock.php

Lines 969 to 972 in 307f708

'everblock_notifications' => $notifications,
'everblock_form' => $this->renderForm(),
'display_upgrade' => $displayUpgrade,
'everblock_stats' => $this->getModuleStatistics(),

P1 Badge Provide stats before rendering form

The new statistics tab fetches views/templates/admin/config/docs/stats.tpl inside renderForm(), but everblock_stats is only assigned afterwards. Because the Smarty variable is missing during template rendering, all counters resolve to zero regardless of actual data. Populate everblock_stats before invoking renderForm() (or pass the data into the form generation) so the template receives the computed counts.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting

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.

1 participant