Unable to switch off Email notifications for an Ad

Sefi · October 21st, 2007, 02:18 PM

Catch another annoying one: (v2.6.1)

In comments.php line 115

Code:

Content visible to verified customers only.

Should look like this

Code:

Content visible to verified customers only.

Without this fix, users are unable to switch of email notifications for an Ad, from the menu link on the Ad display page.

Chuck S · October 21st, 2007, 06:04 PM

I will check it out and let you know but I think that is incorrect. notifyid should be a users userid not an insert id. I beleive you might want to look at this.

http://www.photopost.com/forum/showp...6&postcount=18

Sefi · October 21st, 2007, 07:44 PM

There is another explanation
------------
There is an error in showproduct.php line 614 (it should be SELECT userid)

Code:

Content visible to verified customers only.

this explains why did I think that the notifyid is a unique identifier of a given notification.

Which one is correct ?

Chuck S · October 21st, 2007, 08:23 PM

There is no error in showproduct.php any change I note is in the thread I linked and I beleive in comments.php code should be as I posted.

Sefi · October 22nd, 2007, 05:01 AM

Chuck, I'm sure you are right, but then please explain this in comments.php line 107

Code:

Content visible to verified customers only.

And in install.php

Code:

Content visible to verified customers only.

In showproduct.php this id is used to create the link for switching off ad notification....

Chuck S · October 22nd, 2007, 10:33 AM

Thats just the notification autoincrement field.

If a user wants to DELETE a subscription the query is indeed valid and correct we delete it based on the userid of the user and the category or the product itself depending on whether its a cat subscription or if a user is insubscribing from an ad.

Any correction noted is the one in my post.