tonyair/silverstripe-fulltextsearch
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-fulltextsearch synced 2024-10-22 14:05:29 +02:00
Code Issues Projects Releases Wiki Activity

FIX Don't assume a DataObject's table.

Browse Source 
Knowing if a DataObject has a table or not is irrelevant - data is queried
out and pushed into solr all the same, whether there is a join table, a
base table, or any combination with maybe a few no-tables in between.
SomeDataClass::get() will fetch all data irrelevant of tables, that's the
point of an abstraction like SilverStripe's ORM.
This commit is contained in:
Dylan Wagstaff 2018-02-23 17:37:44 +13:00
parent 10344580c9
commit 9e32f2adb0
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Search/Indexes/SearchIndex.php
View File

@ -282,10 +282,6 @@ abstract class SearchIndex extends ViewableData
throw new Exception('Can\'t add class to Index after fields have already been added'); throw new Exception('Can\'t add class to Index after fields have already been added');
} }
if (!DataObject::getSchema()->classHasTable($class)) {
throw new \InvalidArgumentException('Can\'t add classes which don\'t have data tables (no $db or $has_one set on the class)');
}
$options = array_merge(array( $options = array_merge(array(
'include_children' => true 'include_children' => true
), $options); ), $options);