Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use SYS_LASTCHANGED for latest changes on pages #52

Closed
seirerman opened this issue Aug 17, 2015 · 1 comment
Closed

Use SYS_LASTCHANGED for latest changes on pages #52

seirerman opened this issue Aug 17, 2015 · 1 comment
Labels

Comments

@seirerman
Copy link
Contributor

Right now, pages get indexed with the tstamp value as "changed" in the Solr index.
This means that only changes to the page record itself are considered, and facets for the age of pages are basically worthless if the actual content is not taken into account.

Wouldn't it be better if SYS_LASTCHANGED is used, since it considers both changes to pages and tt_content?

Editing

$document->setField('changed',     $pageRecord['SYS_LASTCHANGED']);

in /typo3conf/ext/solr/Classes/Typo3PageIndexer.php, line 242 solved the problem for me.
I'm not sure if that was all there was to do.

@irnnr irnnr added the bug label Aug 18, 2015
@irnnr
Copy link
Collaborator

irnnr commented Aug 18, 2015

Good point, the record monitor is taking that into account actually. Will have to look into that.

@irnnr irnnr closed this as completed in 274c916 Sep 26, 2015
irnnr added a commit that referenced this issue Sep 26, 2015
Fixes #52

Change-Id: Ia08a23852180247be5f3ef8d853a1b7ec2ef7b48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants