Show the "my quotes" menu item - not working

bekasu - September 22, 2008 - 03:06
Project:Quotes
Version:6.x-1.1-alpha5
Component:Code
Category:bug report
Priority:normal
Assigned:NancyDru
Status:closed
Description

I've tried this in the stable 6.1 version and the development version.

Nothing changes when I check or uncheck the "Show the 'My Quotes' menu item option on the Aministration>>Quote Settings>>My Quote Links section.

The Administration>>Quotes Settings>>My Quote Links>>Show link to users' quotes -- this check/uncheck does work.

I was expecting to have a navigation link show up in the navigation bar which would display the quotes for the authorized user who is logged in.

My permissions are: create quotes and edit own quotes as activated .. other three options are not activated.

#1

NancyDru - September 22, 2008 - 04:18

Try clearing the cache (admin >> site config >> performance).

#2

bekasu - September 23, 2008 - 01:42

Tried (admin>>site config>>performance>>clear cache)

no change.

Tried:
Admin>quote settings> 1) uncheck my quotes & 2) check recent quotes on my account page

(admin>>site config>> performance>clear cache
result: recent quotes link appears correctly & links correctly

Tried:
Admin>>quote settings>> checked my quotes
admin>>sig config>>performance>clear cache
result: no change in navigation bar

I'm not seeing a menu item show up ... shouldn't it be in the navigation bar?

#3

NancyDru - September 23, 2008 - 04:01

It should be in the "Navigation" menu, yes. If you move Quotes to a different menu, such as Primary links, it will probably not show up.

#4

bekasu - September 24, 2008 - 01:54

Okay,
I have a navigation bar entry called Quotes with the path of localhost/quotes

If i uncheck the 'my quotes' box on the Admin>>Site Config>>Quote Settings, there is no change in the navigation bar.

Tried:
Uninstall of quotes module.
Navigation bar lost the quotes link (good)
menu_router table had no quotes entries
system table had the following entry left: sites/all/modules/quotes/quotes.module

Tried:
removing the left over entry in the system table
reinstall quotes module
the navigation bar has the quotes link back with the link localhost/quotes

If that is the correct behavior then it is working; however, I expected a 'my quotes' link to 1) actually say 'my quotes' and 2) link only to my quotes. This does neither.

Not trying to be a pest, I'm just trying to help/understand.

#5

NancyDru - September 24, 2008 - 04:04

I guess I'll have to rework the menu. It got quirky in 6.x.

#6

NancyDru - September 27, 2008 - 17:45
Assigned to:Anonymous» NancyDru
Status:active» fixed

I reworked the menu access so that it should work now. Please test the -dev version when it rolls out this evening.

#7

bekasu - September 28, 2008 - 02:14

Doesn't work for me yet.

Here is what i did:
1. disable Quotes module
2. Uninstall quotes module
3. delete sites>>all>>modules>>quotes directory
4. download new code
5. verify that something has changed (quotes.module has a new datestamp)
6. put quotes module code in sites>>all>>module directory
7. enable quotes module
8. ensure that administer>>quote settings>>show my quotes is enabled
9. check on user permissions & that the quotes_show_myquotes variable in the variable table was set to TRUE (that is, 1)
10. used firefox as admin on localhost, used IE as test user... neither id has a my quotes link, both id's can see each other's quotes

What I expected .. an entry for My Quotes in the navigation bar OR the Quotes link in the navigation bar only shows quotes for the logged on user

There is no My Quotes link--
The Quotes link shows all quotes

I'll keep looking to see if I can figure out the problem, although I'll be the first to admit I'm a novice drupal debugger.. about the best I can tell you at this point is i THINK the problem is at line 1018 in the quotes.module.. or whatever occurs prior to executing line 1018.. that is, the $account value is not set when it gets there so it falls through the loop but again, I'm still learning this new world.

#8

NancyDru - September 28, 2008 - 11:44

If it is not set, then it's the fault of the menu system.

#9

NancyDru - September 28, 2008 - 11:44
Status:fixed» postponed (maintainer needs more info)

#10

NancyDru - February 10, 2009 - 14:38
Status:postponed (maintainer needs more info)» fixed

Fix committed for 6.x (not needed in 5.x)

#11

System Message - February 24, 2009 - 14:40
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.