XGUI does not show CAMI Privileges that was registered after XGUI initialize #64

Open
roboderpy opened this Issue Jul 20, 2016 · 3 comments

Projects

None yet

3 participants

@roboderpy
roboderpy commented Jul 20, 2016 edited

EDIT: It does not show registered CAMI Privileges that was registered after XGUI is initialized

Steps to reproduce

Register CAMI Privilege after XGUI is initialized

Expected behavior

  1. You are able to restrict access per group
  2. Access appears in XGUI group configuration menu

Actual behavior

Only first

Error(s) in server console, if any

None

Error(s) in player's console, if any

None

Version

] ulx version
ULib v2.61d
ULX v3.71d

Screenshots

2016-07-20_15-03_garry s mod
2016-07-20_15-47_garry s mod
https://github.com/roboderpy/dpp/blob/master/lua/dpp/sh_access.lua#L29

Suggestion about current menu

In the hover hint please display full Privilege name instead of just "A privilege from CAMI"
2016-07-20_15-53_garry s mod

@roboderpy roboderpy changed the title from XGUI does not show registered CAMI Privileges to XGUI does not show CAMI Privileges that was registered after XGUI initialize Jul 20, 2016
@Nayruden
Member

@SticklyMan, do we need to make some kind of hook to notify XGUI when privileges change?

@SticklyMan
Contributor

@Nayruden The issue is likely here, which only happens on ulx.HOOK_ULXDONELOADING:
https://github.com/TeamUlysses/ulx/blob/master/lua/ulx/xgui/server/sv_groups.lua#L301-L309

Does UCLChanged get called when a privilege is changed? Otherwise, yeah, XGUI will need some kind of notification to repopulate and send out the updated accesses.

@Nayruden Nayruden self-assigned this Jul 21, 2016
@Nayruden
Member

Got it; taking on the task of creating a hook for new privileges. @SticklyMan, you can implement it on your side (XGUI) before me; just tell me the hook name you choose.

@roboderpy roboderpy referenced this issue in roboderpy/dpp Aug 9, 2016
Closed

list of what each CAMI convar does? #14

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