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.

Sign up for GitHub

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

Jump to bottom

Menu for non-admins broken #297

Open
yafp opened this issue Oct 23, 2019 · 2 comments
Open

Menu for non-admins broken #297

yafp opened this issue Oct 23, 2019 · 2 comments
Assignees
@yafp
Labels
Cat: bug
Milestone
4.1.0

Comments

@yafp
Copy link
Owner

yafp commented Oct 23, 2019

The menu for non-admin account throws an error

Notice
: Undefined index: admin in
/var/www/f/inc/genericNavigation.php
on line 24

This is displayed between 'Keyboard' and 'Logout'
@yafp yafp added the Cat: bug label Oct 23, 2019
@yafp yafp added this to the Version 5 (Pentagonum) milestone Oct 23, 2019
@yafp
Copy link
Owner Author

yafp commented Oct 23, 2019

should use isset

@yafp yafp self-assigned this Oct 23, 2019
@yafp
Copy link
Owner Author

yafp commented Dec 20, 2019

While login the session variable

$_SESSION[ 'monoto' ][ 'admin' ]

is only set if the user is admin (with a value of 1.

Possible fix:
should set the session variable for non-admins as well - with a value of 0

@yafp yafp modified the milestones: Version 5 (Pentagonum), 4.1.0 Dec 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yafp yafp

Labels
Cat: bug
Projects
None yet
Milestone
4.1.0
Development

No branches or pull requests
1 participant
@yafp