MightyGorgon / icy_phoenix_plugins

Plugins for Icy Phoenix
http://www.icyphoenix.com
7 stars 6 forks source link

[!][Cash] The config links don't have SID #29

Closed vendethiel closed 7 years ago

vendethiel commented 9 years ago

major breakage

Basically, the links in "cash management" have NO SID. So when you try to click them, you get redirected to IP's index instead...

HOWEVER the topbar menu works correctly.

vendethiel commented 9 years ago

Also, they're not in the plugins folder, but in the IP_ROOT/adm/ folder, it should get fixed at some point.

MWE001 commented 7 years ago

I think the dirty fix that Mort suggested works just fine. Is there really a need to have a nav bar up top and Menu horizontally as well on the very same page? That is kind of redundant.

vendethiel commented 7 years ago

@MWE001 I'd even suggest we enable the ACP menu mode by default, and remove the option to switch it off..

MWE001 commented 7 years ago

Yeah that would actually work better in my opinion.I don't mind the admin nav for FAP but on the cash mod it has always kind of bugged me (no pun intended).

MWE001 commented 7 years ago

In order to do that though, one must do this fix that Luca gave to me in this thread http://www.icyphoenix.com/viewtopic.php?f=2&t=9623 Without that fix, the whole ACP menu is garbage. The links all went back to 404 pages. With The fix Luca gave me, my ACP menu works a treat.

vendethiel commented 7 years ago

MG is ahead of you :P

MWE001 commented 7 years ago

Nice! And actually thinking about it man, the cash mod has not 1, not 2 but 3 menus! lol We could eliminate 2 of them an d only have one as it should be.

vendethiel commented 7 years ago

yes, I agree.

MWE001 commented 7 years ago

I attempted to work on this tonight and all kinds of things went wrong real fast. I was able to remove the switch in the acp, and after that thing started going horribly wrong. I don't know where the switch is in the php file and how to set the acp up default to navbar instead of menu. I reverted my files back to morts quick dirty fix.