This is a minor improvement.
The fix is straightforward.
In sparql_entity_storage module, in sparql_entity_storage_form_alter there is the following code:
foreach ($base_fields as $field_name => $base_field) {
// The entity id doesn't need a mapping as it's the subject of the triple.
if ($field_name === $id_key) {
continue;
}
$columns = $base_field->getColumns();
foreach ($columns as $column_name => $column) {
$title = $base_field->getLabel();
if (count($columns) > 1) {
$title .= ' (' . $column_name . ')';
}
$form['sparql_entity_storage']['base_fields_mapping'][$field_name] = [
'#type' => 'details',
'#title' => $title,
'#description' => $base_field->getDescription(),
];
$has_mapping = $mapping && $mapping->getMapping($field_name, $column_name);
$form['sparql_entity_storage']['base_fields_mapping'][$field_name][$column_name]['predicate'] = [
As it is quite clear by the code itself, the 'details' fieldset is built for each column mainly overriding the previous property, thus only the last property shown.
Issue fork sparql_entity_storage-3271100
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
dimilias commentedComment #4
claudiu.cristeaChanges look good. However the tests are not green (coding standards issues).
Comment #5
dimilias commentedComment #6
claudiu.cristeaLooks good.
Comment #8
claudiu.cristeaThank you