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.

Command icon 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

idimopoulos created an issue. See original summary.

dimilias’s picture

Assigned: dimilias » Unassigned
Status: Active » Needs review
claudiu.cristea’s picture

Status: Needs review » Needs work

Changes look good. However the tests are not green (coding standards issues).

dimilias’s picture

Status: Needs work » Needs review
claudiu.cristea’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

claudiu.cristea’s picture

Status: Reviewed & tested by the community » Fixed

Thank you

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.