Make that on any edit screen (i.e. User Manager: Edit Group, Add User Note...etc)

So, the edit views need the JFactory::getApplication()->input->set('hidemainmenu', true); line like the CMS normally uses. And if they do have it, then we need to look at mod_menu again to see WTF is going on.

I'll take a look at that. Thanks, I appreciate you sharing that knowledge.

Best,

Matt Thomas
Founder betweenbrain http://betweenbrain.com/™
Lead Developer Construct Template Development
Frameworkhttp://construct-framework.com/
Phone: 203.632.9322
Twitter: @betweenbrain
Github: https://github.com/betweenbrain

On Thu, Dec 20, 2012 at 12:18 AM, Michael Babker
notifications@github.comwrote:

So, the edit views need the JFactory::getApplication()->input->set('hidemainmenu',
true); line like the CMS normally uses. And if they do have it, then we
need to look at mod_menu again to see WTF is going on.

—
Reply to this email directly or view it on GitHubhttps://github.com/JTracker/jissues/issues/50#issuecomment-11560662.

Looks like it's there, for example starting at
https://github.com/JTracker/jissues/blob/master/administrator/components/com_categories/views/category/view.html.php#L37

and is in mod_menu
https://github.com/JTracker/jissues/blob/master/administrator/modules/mod_menu/mod_menu.php#L27

Best,

Matt Thomas
Founder betweenbrain http://betweenbrain.com/™
Lead Developer Construct Template Development
Frameworkhttp://construct-framework.com/
Phone: 203.632.9322
Twitter: @betweenbrain
Github: https://github.com/betweenbrain

On Thu, Dec 20, 2012 at 8:48 AM, Matt Thomas matt@betweenbrain.com wrote:

I'll take a look at that. Thanks, I appreciate you sharing that knowledge.

Best,

Matt Thomas
Founder betweenbrain http://betweenbrain.com/™
Lead Developer Construct Template Development Frameworkhttp://construct-framework.com/
Phone: 203.632.9322
Twitter: @betweenbrain
Github: https://github.com/betweenbrain

On Thu, Dec 20, 2012 at 12:18 AM, Michael Babker <notifications@github.com

wrote:

So, the edit views need the JFactory::getApplication()->input->set('hidemainmenu',
true); line like the CMS normally uses. And if they do have it, then we
need to look at mod_menu again to see WTF is going on.

—
Reply to this email directly or view it on GitHubhttps://github.com/JTracker/jissues/issues/50#issuecomment-11560662.

So I'm thinking we should look at and see if that $enabled flag is being watched?

Was just about to look at that and got sidetracked with something else.
Will do in a bit.

Best,

Matt Thomas
Founder betweenbrain http://betweenbrain.com/™
Lead Developer Construct Template Development
Frameworkhttp://construct-framework.com/
Phone: 203.632.9322
Twitter: @betweenbrain
Github: https://github.com/betweenbrain

On Thu, Dec 20, 2012 at 9:20 AM, Chad Windnagle notifications@github.comwrote:

So I'm thinking we should look at and see if that '''$enabled''' flag is
being watched?

—
Reply to this email directly or view it on GitHubhttps://github.com/JTracker/jissues/issues/50#issuecomment-11574983.

Hmm, I should get the mvc branch in a mergeable state.. I can not confirm this bug on this branch - any chance you could check it out ?
That said - it's a bit different and also needs a different install.sql (which may still have some differences) - just treat it as WIP ;)

Yes, I'll check for this on the MVC branch. Thanks!

Best,

Matt Thomas
Founder betweenbrain http://betweenbrain.com/™
Lead Developer Construct Template Development
Frameworkhttp://construct-framework.com/
Phone: 203.632.9322
Twitter: @betweenbrain
Github: https://github.com/betweenbrain

On Thu, Dec 20, 2012 at 12:03 PM, Nikolai Plath notifications@github.comwrote:

Hmm, I should get the mvc branch in a mergeable state.. I can not confirm
this bug on this branch - any chance you could check it out ?
That said - it's a bit different and also needs a different install.sql
(which may still have some differences) - just treat it as WIP ;)

—
Reply to this email directly or view it on GitHubhttps://github.com/JTracker/jissues/issues/50#issuecomment-11581452.

I have confirmed this bug on the MVC branch. Try navigating to /trkadmin/index.php?option=com_users&view=user&layout=edit&id=1 and you will see something like the screenshot below.

From what I can see - the menu is actually disabled..
Or are you talking about the "wording" ? - that's actually true - the disabled menu displays "nonsense" - confirmed ;)

Though I do not get this notice.

I guess that is a better way to describe it:

the disabled menu displays "nonsense"

Best,

Matt Thomas
Founder betweenbrain http://betweenbrain.com/™
Lead Developer Construct Template Development
Frameworkhttp://construct-framework.com/
Phone: 203.632.9322
Twitter: @betweenbrain
Github: https://github.com/betweenbrain

On Thu, Dec 20, 2012 at 12:55 PM, Nikolai Plath notifications@github.comwrote:

that's actually true - the disabled menu displays "nonsense"

Heh, that makes sense... LOL

On Thursday, December 20, 2012, Matt Thomas wrote:

I guess that is a better way to describe it:

the disabled menu displays "nonsense"

Best,

Matt Thomas
Founder betweenbrain http://betweenbrain.com/™
Lead Developer Construct Template Development
Frameworkhttp://construct-framework.com/
Phone: 203.632.9322
Twitter: @betweenbrain
Github: https://github.com/betweenbrain

On Thu, Dec 20, 2012 at 12:55 PM, Nikolai Plath >wrote:

that's actually true - the disabled menu displays "nonsense"

—
Reply to this email directly or view it on GitHubhttps://github.com/JTracker/jissues/issues/50#issuecomment-11583745.

• Michael

Please pardon any errors, this message was sent from my iPhone.

Note to self after re-reading screenshot... The sooner admin application gets refactored off legacy, the better. Do we really need that JPATH_INSTALLATION define!?

+1

Best,

Matt Thomas
Founder betweenbrain http://betweenbrain.com/™
Lead Developer Construct Template Development
Frameworkhttp://construct-framework.com/
Phone: 203.632.9322
Twitter: @betweenbrain
Github: https://github.com/betweenbrain

On Thu, Dec 20, 2012 at 3:16 PM, Michael Babker notifications@github.comwrote:

Note to self after re-reading screenshot... The sooner admin application
gets refactored off legacy, the better. Do we really need that
JPATH_INSTALLATION define!?

—
Reply to this email directly or view it on GitHubhttps://github.com/JTracker/jissues/issues/50#issuecomment-11589110.

Well, the admin application already is "off legacy" (in the mvc branch) except "some stuff" - that is: com_categories and com_config.
And then there is: https://github.com/JTracker/jissues/blob/mvc/libraries/tracker/application/admin.php

I've added the JPATH_INSTALLATION constant to the admin/defines with a @legacy note until we finish refactoring those components ;)