The following warnings occurred:
Warning [2] Undefined variable $unreadreports - Line: 26 - File: global.php(961) : eval()'d code PHP 8.2.25 (Linux)
File Line Function
/global.php(961) : eval()'d code 26 errorHandler->error
/global.php 961 eval
/printthread.php 16 require_once
Warning [2] Undefined property: MyLanguage::$archive_pages - Line: 2 - File: printthread.php(287) : eval()'d code PHP 8.2.25 (Linux)
File Line Function
/printthread.php(287) : eval()'d code 2 errorHandler->error
/printthread.php 287 eval
/printthread.php 117 printthread_multipage



UserSpice
Status Update 028 - Progress - Printable Version

+- UserSpice (https://userspice.com/forums)
+-- Forum: Support Center (https://userspice.com/forums/forumdisplay.php?fid=23)
+--- Forum: UserSpice 4.3 and Below (https://userspice.com/forums/forumdisplay.php?fid=26)
+--- Thread: Status Update 028 - Progress (/showthread.php?tid=775)

Pages: 1 2 3 4 5 6 7 8

Status Update 028 - Progress - Brandin - 09-26-2017

Will take a look when I get home and I will push your changes to a Git branch for @mudmin Smile

Status Update 028 - Progress - Brandin - 09-26-2017

Alright!

I made a few changes.

I included the setting to toggle navigation and changed the files around to match this.

With this: I noticed on the old style, we screwed up the styling, so I made the CSS inline only if using the db-driven nav.

BUILD NOTES STATUS 28

UPDATES
users
helpers
class.treeManager.php - corrected Line 80 per Karsen
us_helpers.php - corrected checkMenu class per Karsen
language.php - added language keys from Nav that were removed

css
sb-admin.css - Karsens corrections - added to this

includes
navigation.php - reversed to default, added an if for the nav type
database-navigation.php - added two classes, navbar-noborder and re-added navbar-inverse

profile.php - Fixed page per Karsen
admin.php - Updated to remove PHP for css_sample - per Karsen, added PHP for new setting
_admin_site_settings.php - Added HTML for new setting
admin_menu.php - There was a double call for die() on securePage, removed

SQL
ALTER TABLE
Code:
groups_menus
CHANGE
Code:
id
Code:
id
INT(11) NOT NULL AUTO_INCREMENT;
ALTER TABLE
Code:
settings
ADD
Code:
navigation_type
INT(1) NOT NULL DEFAULT '1' AFTER
Code:
page_default_private
;

Status Update 028 - Progress - Brandin - 09-26-2017

OH....

Why don't we have a redirect on maintenance.php in case offline = 0?!?!?!?!

I can just imagine someone has that ONE user that keeps refreshing that page and they ain't gettin nowhere!!!! (is it just me giggling because I know the type of people I work with?!?!?! lol)

if($settings->site_offline==0) Redirect::to('index.php');

Status Update 028 - Progress - mudmin - 09-27-2017

Oops. Yeah. We need that.

Status Update 028 - Progress - karsen - 09-29-2017

Viewing either admin_menu.php or admin_menu_item.php without a GET variable will display all kinds of errors. Instead we should either display a message with a link to admin_menus.php or simply redirect them there.

Status Update 028 - Progress - mudmin - 09-29-2017

Yep. Totally agreed. We just need to redirect to admin_menus

Status Update 028 - Progress - mudmin - 09-30-2017

Do either of you guys @Brandin or @Karsen have updates?

Status Update 028 - Progress - Brandin - 09-30-2017

Unfortunately my internet suddenly went down and can't be fixed until Monday so no I do not. Thanks for checking.

Status Update 028 - Progress - karsen - 10-02-2017

Not right now. I think I'll have a little free time in the next couple days, so if you update to 029 I'll grab it and test our changes, plus see what might need some polish. I also want to add in page descriptions for each page that will display in the html
Code:
title
tag, instead of the simple and generic
Code:
UserSpice
. It was a user request from a few weeks back that I forgot about until this weekend.

Status Update 028 - Progress - Brandin - 10-02-2017

Agreed - internet is back up now and I have free time this week so I'd like to get testing and think of some polishing touches