Index author when node access is enabled [#1324182]

When you enable node access on an index, currently the published field is forced to be index, but the uid/author is required as well.

Comment	File	Size	Author
#2	1324182--index-author-for-node-access-2.patch	2.55 KB	drunken monkey
#2
#1	1324182-force-index-author.patch	1.06 KB	dawehner
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

dawehner

German

CreditAttribution: dawehner commented 27 October 2011 at 21:34

File	Size
1324182-force-index-author.patch	1.06 KB

Here is the patch. In general i'm not sure whether it's required to index both author and uid.

Comment #2

drunken monkey

he/him

German

Vienna, Austria

CreditAttribution: drunken monkey commented 28 October 2011 at 16:34

Title:	Index uid/author beside published when you enable node access	» Index author when node access is enabled
Component:	Documentation	» Plugins

File	Size
1324182--index-author-for-node-access-2.patch	2.55 KB

Thanks for catching that, you're right of course! But now I really checked, nothing else should be required.

We only need either uid or author, depending on what we use for the query. I now opted for "author", as it is considerably more useful for other purposes, and thus more likely to be indexed in any case. Thanks for that suggestion, too!
In theory, we could dynamically check if either of the fields is indexed and use that one, but in practice I think there's really no benefit in indexing "uid" anyways.

Patch attached, using "author".