ok.. I did... sorry for my oversight. it is not behaving like it is supposed to behave but still works pretty well. the normal behavior is to chow the link to "review the listing" but when clicked show the "you need to register message" The behavior resulting from adding the code in blue is that if not logged in, it does not shoe the link to leave review. if you are logged in, you do see the link to leave review.
Huh?? If they cant write a review, why do you even want to allow them to be able to click a link? Just so you can tell them to log in?? Doesnt make any sense. If they need to be logged into write a review, then dont waste their time by even allowing them to click a link thinking they can write a review but only tell them they need to log in. Thats like going to a restaurant and them handing you a breakfast menu and a lunch menu. Then you look through the breakfast menu, decide you want an omlette and when the waitress gets to you to take your order says "Im sorry we arent service breakfast".
I understand what you are saying. This is why the initial intention of the module is to ask them to log in. This is in turn will help convert visitors into members.
The intention of the module is to allow people to leave feedback. You can limit their ability to provide feedback or a rating by adding code to check if they are logged in. The directions for the module do not state the user will be told to login once the review link is clicked. It simply states you can limit who writes a review to those who are logged in. Wasting someones time and showing them a link they cant use just to tell them to log in does not turn them into users/members any more than simply displaying a message to login to write a review does. I would even argue it turns people away from your site because you are wasting their time. If you want users, then think about things from their perspective and not your own. Make the site easy and respectful to them and users will come. Waste their time having them click stuff just to tell them they have to login and register and you piss people off.
I respect your opinion in this subject and most definitely respect your knowledge and experience coding and programming. I on the other hand think different. Having spent most of my live dedicated to marketing and advertising in multiple industries. Giving someone a partial taste of what they can do in the site, more likely than not will convert visitors into members. Its like showing them foreclosure properties available but then requiring them to register in order to view the contact details (or any other detail for that matter) Thanks again for all your help resolving the various problems i encountered installing this module. You are awesome and a very valuable member of the forum!
When someone sees text that states "Log in to leave a review" they ARE getting a taste for what can be done by joining the site. Having them click a link and then showing them a message also gives them a taste but it teases them as well. Wasting their precious time having them click something only to tell them what you could have told just as easily without wasting their time. If I goto a site and they pull that crap with me I no longer use the site. If I goto a diner and they give me a breakfast menu and later when I goto order tell me they arent serving me breakfast, they dont encourage me to come back for breakfast, on the contrary, I get up and leave because they just wasted my time. Its your site, you can do what ever you want but I know I am not alone in my thinking. You can gently encourage people to sign up for your site without being rude to them. If you want to alter things to work as you describe then you are going to have to alter the template that actually displays the form and place the whole form within the logical expression where you now just have the link.