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

Update access.php #111

Conversation

deepend-tildeclub
Copy link
Contributor

here is the error you'd get before the fix.. you would find the error on the create a poll page.

Warning: Undefined array key "btAccessCache" in /home/retrodig/public_html/zarksfallenangels/classes/access.php on line 102

deepend-tildeclub and others added 3 commits January 24, 2024 15:11
here is the error you'd get before the fix.. you would find the error on the create a poll page. 

Warning: Undefined array key "btAccessCache" in /home/retrodig/public_html/zarksfallenangels/classes/access.php on line 102
When on the create a poll page. 

Fatal error: Uncaught TypeError: count(): Argument RedDragonWebDesign#1 ($value) must be of type Countable|array, null given in /home/retrodig/public_html/zarksfallenangels/classes/access.php:279 Stack trace: #0 /home/retrodig/public_html/zarksfallenangels/members/include/polls/createpoll.php(299): Access->dispSetMemberAccess() RedDragonWebDesign#1 /home/retrodig/public_html/zarksfallenangels/members/console.php(150): require_once('...') RedDragonWebDesign#2 {main} thrown in /home/retrodig/public_html/zarksfallenangels/classes/access.php on line 279
src/classes/access.php Outdated Show resolved Hide resolved
changed array() to []
src/classes/access.php Outdated Show resolved Hide resolved
removed unnecessary  lines.   Tested and the error didn't come back .  So guess I over thought things.
@RedDragonWebDesign RedDragonWebDesign merged commit 3088b3d into RedDragonWebDesign:master Jan 25, 2024
1 check passed
@deepend-tildeclub deepend-tildeclub deleted the initialize-session-array-to-fix-error-in-classes/access.php branch February 1, 2024 02:48
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.

2 participants