I get the following error:
Notice: Undefined variable: user in nice_menus_block_view() (line 254 of C:\wamp\www\seven\sites\all\modules\nice_menus\nice_menus.module).
Notice: Trying to get property of non-object in nice_menus_block_view() (line 254 of C:\wamp\www\seven\sites\all\modules\nice_menus\nice_menus.module).

CommentFileSizeAuthor
#2 1017618-nice_menus-d7.module.patch671 bytesjcarlson34
#1 nice_menus-[1017618].module.patch670 bytesjcarlson34

Comments

jcarlson34’s picture

jcarlson34 commented
Status: Active » Needs review
StatusFileSize
new nice_menus-[1017618].module.patch670 bytes

Looks like this might be an easy fix.

All this patch does is change line 254's if ($output['subject'] == t('Navigation') && $user->uid) { from 'Navigation' to 'navigation'.

The field menu_name in the menu_custom table has the menus listed by lowercase.

jcarlson34’s picture

jcarlson34 commented
StatusFileSize
new 1017618-nice_menus-d7.module.patch671 bytes

Shoot. Added a patch for HEAD. Here's a patch for 7.x-2.0-beta1 instead...

jlporter’s picture

jlporter commented

Also have the same notice. Please roll into new beta?

add1sun’s picture

add1sun commented
Status: Needs review » Fixed

Committed to HEAD. Thanks!

jcarlson34’s picture

jcarlson34 commented

Cool my first ever patch was committed. Woot! :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

as2’s picture

as2 commented

As I have translations on my page, this does not work
if (($output['subject'] == t('Navigation')) && $user->uid)

this is fine tough
if (($output['subject'] == 'Navigation') && $user->uid)

Michael.Kling’s picture

Michael.Kling commented
Version: 7.x-2.0-beta1 » 7.x-2.0
Status: Closed (fixed) » Active

Had this issue with Nice Menus 7.x-2.0 and German translation active

I could solve it with a merge of as2's solution:
Changing
if ($output['subject'] == t('navigation') && $user->uid) {
to
if ($output['subject'] == 'navigation' && $user->uid) {

astonvictor’s picture

astonvictor commented
Issue summary: View changes
Status: Active » Closed (outdated)

I'm closing it because the issue was created a long time ago without any further steps.

if you still need it then raise a new one.
thanks