Download & Extend

Overrides search listing by presenting password form

Project:Protected node
Version:6.x-1.5
Component:Code
Category:bug report
Priority:normal
Assigned:AlexisWilke
Status:closed (fixed)

Issue Summary

When a site search (search.module) is performed and a Protected Node is returned as part of the search, rather than displaying the list of nodes, protected_node fires off a redirect to the Password Protection---this prevents *any* search results from being returned.

One should be able to perform a search and have results returned without triggering the password.

Comments

#1

I guess this shouldn't happen because no content is returned for the page when it is fetched from a cron run (although this doesn't work well), so that no content should ever be indexed. Hmm.

#2

Same issue here. Subscribing.

#3

Status:active» needs review

I have made a patch, which fix the issue when trying to see search results when some of the nodes is password protected.

AttachmentSize
protected_node_search_result.patch 794 bytes

#4

Status:needs review» reviewed & tested by the community

The patch in #3 works.. a password protected node will still show the title but no text.

If you want to exclude the nodes from search results restricted search will offer a all or nothing solution

#5

Status:reviewed & tested by the community» needs work

Interesting problem. 8-)

The search is fed empty data when the password is set on a node. However, if you create a node, the search indexes it, and then you add a password to that node, it obviously is going to be in the search index, no matter what.

Thinking about it, it seems that the db_rewrite_sql() is the solution for this one (and many others...)

See: #830016: db_rewrite_sql() misses protected nodes...

Thank you.
Alexis

#6

Version:6.x-1.4» 6.x-1.5
Assigned to:Anonymous» AlexisWilke

Okay, the hook_init() was the solution on this one. The other things did not directly help.

It is fixed in 6.x-1.x for now. I'll wait a little before publishing 1.6.

Thank you.
Alexis Wilke

#7

Status:needs work» fixed

Meant to mark as fixed. 8-)

#8

Status:fixed» closed (fixed)

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

nobody click here