Currently the action entity is provided by the system module, but the ::uri() method points to a path defined just by action module.

Let's fix that.

CommentFileSizeAuthor
#1 action-2020985-1.patch2.05 KBdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs review
FileSize
2.05 KB

Here is a patch for it.

alansaviolobo’s picture

Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs reroll
thtas’s picture

Assigned: Unassigned » thtas
thtas’s picture

Assigned: thtas » Unassigned
Status: Needs work » Needs review

Looks like the action module has changed significantly since this original issue was created and this may have already been resolved.
Could somebody with more knowledge of this please confirm?

rishikant05’s picture

rishikant05’s picture

Assigned: Unassigned » rishikant05

patch is not applied.

rishikant05’s picture

Assigned: rishikant05 » Unassigned
star-szr’s picture

Status: Needs review » Closed (duplicate)
Issue tags: -Needs reroll
Related issues: +#2083615: Use links annotation for config entity URI like for content entities

When this issue was opened, Drupal\system\Plugin\Core\Entity\Action contained this code:

  /**
   * {@inheritdoc}
   */
  public function uri() {
    return array(
      'path' => 'admin/config/system/actions/configure/' . $this->id(),
      'options' => array(
        'entity_type' => $this->entityType,
        'entity' => $this,
      ),
    );
  }

As far as I can tell, #2083615-55: Use links annotation for config entity URI like for content entities ultimately fixed this by removing the edit-link definition from the system.module-provided Action entity to action.module. And HEAD still has it like this. So closing as a duplicate.

If I'm missing a clue somewhere please reopen.