Skip to content

[Fix]: Restrict mode + warniongs and errors in logs - #109

Open
jdurand-teclib wants to merge 2 commits into
mainfrom
fix/restrict-mode
Open

[Fix]: Restrict mode + warniongs and errors in logs#109
jdurand-teclib wants to merge 2 commits into
mainfrom
fix/restrict-mode

Conversation

@jdurand-teclib

Copy link
Copy Markdown
  • I have performed a self-review of my code.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes a bug where restrict mode was broken
  • Here is a brief description of what this PR does
    • Remove deprecated usage of front/category.php and front/category.form.php
    • Remove variable assignation by reference in hook.php
    • Fix restrict mode broken because of:
      • A missing alias in a SQL request that was later silently making PHP refer to an unknown index in an array
      • A missing filtering in a SQL request that was supposed to filter group according to strict mode enablement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant