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.

Sign up for GitHub

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

Jump to bottom

[BUG] No indexation for record with starttime in the future #4263

Closed
marc-munos opened this issue Dec 12, 2024 · 2 comments
Closed

[BUG] No indexation for record with starttime in the future #4263

marc-munos opened this issue Dec 12, 2024 · 2 comments

Comments

@marc-munos
Copy link

Describe the bug
Records with starttime in the future are not queued for indexation

To Reproduce

  • Create a record, a page for instance
  • Set a starttime in the future
  • assert the record is not queued for indexation (ie. not inserted into tx_solr_indexqueue_item)

Expected behavior
The record should be inserted in queue with a changed date equal to the starttime

Screenshots
na

Used versions (please complete the following information):

  • TYPO3 Version: 12
  • EXT:solr Version: 12

Additional context
Add any other context about the problem here.
marc-munos added a commit to TechnoQuebecca/ext-solr that referenced this issue Dec 12, 2024
@marc-munos
TYPO3-Solr#4263 [FIX] - fixed ConfigurationAwareRecordService.php
1a96dec 
Related to issue 4263 : just reverting the ConfigurationAwareRecordService::getRecordForIndexConfigurationIsValid() method to v11.5.2, relying on Backend::getRecord() method instead of using a queryBuilder() adding constraints preventing to retrieve a not-yet visible record.
The issue could also be fixed by removing restrictions, for instance : $queryBuilder->getRestrictions()
    ->removeByType(StartTimeRestriction::class)
@marc-munos
Copy link
Author

proposed fix : #4264

@dkd-kaehm dkd-kaehm mentioned this issue Dec 17, 2024
Closed
@dkd-kaehm
Copy link
Collaborator

Closing: is fixed in #4169, #4258

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#4263 [FIX] - fixed ConfigurationAwareRecordService.php TechnoQuebecca/ext-solr
2 participants
@dkd-kaehm @marc-munos