This is a follow-up from #1552396: Convert vocabularies into configuration
from /core/modules/taxonomy/lib/Drupal/taxonomy/Plugin/views/filter/TaxonomyIndexTid.php
value_form()
$options = array();
$query = db_select('taxonomy_term_data', 'td');
//$query->innerJoin('taxonomy_vocabulary', 'tv', 'td.vid = tv.vid');
$query->fields('td');
// @todo Sorting on vocabulary weight and name.
//$query->orderby('tv.weight');
//$query->orderby('tv.name');
$query->orderby('td.weight');
$query->orderby('td.name');
$query->addTag('term_access');
if ($this->options['limit']) {
$query->condition('td.vid', $vocabulary->id());
}
$result = $query->execute();
foreach ($result as $term) {
$options[$term->tid] = $term->name;
}
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | add_back_ability_to-1821274-2.patch | 754 bytes | imyaro |
Comments
Comment #1
andypostComment #2
imyaro commentedWe still have comment
As for me - it should be removed if it works as designed.
Comment #3
imyaro commentedComment #7
andypostComment #9
klonos...well, only name should be a matter for this issue, since vocabulary weight is to be deprecated: #3008064: Deprecate vocabulary weight property
Right?
Comment #15
liquidcms commentedWhy is this "as designed"? and why was being able to sort by vocab removed?
Comment #19
quietone commentedClosed #3069616: The ordering of Vocabularies on field settings doesn't follow the order on the vocabulary listing page as a duplicate.
Comment #20
wim leersPer @catch in #3008064-31: Deprecate vocabulary weight property:
Closed that issue in favor of this issue. 👍