Problem/Motivation

Displays of event series and event instances do not contain contextual links (for editing or deleting).

Steps to reproduce

  1. Create some events
  2. Create a view that lists eventinstances in an unformatted list (this is just how I did it, but an entity reference field could be used to view an individual event instance too, though I don't have a use case for that)
  3. Go to a page where you can see a list of those eventinstances
  4. See that there are no contextual links for each list item

Proposed resolution

Add contextual link definitions and fix the templates so that contextual links can be seen.

Issue fork recurring_events-3575563

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:

About issue forks

Comments

megakeegman created an issue. See original summary.

megakeegman opened merge request !188

chrisla’s picture

chrisla commented

I would appreciate if this contextual link was optional, as I do not want to encourage users to edit the event series nor accidentally edit the series instead of the instance. Perhaps there could be a toggle on Event Series Settings (/admin/structure/events/series/settings) that enables or disables this contextual link.

megakeegman’s picture

megakeegman commented

This MR adds edit and delete to both event series and event instance. While the other issue would address a custom contextual link to edit the event series from the event instance.

Should there be a setting for each contextual link?

FWIW, the contextual links only show up for people that have the relevant permissions. So if someone does not have the permission to edit an individual event instance (either via edit own or edit any perms), then those contextual links will not appear for them. Thought it was worth mentioning that some safeguard is already built into contextual links by default through permissions.

chrisla’s picture

chrisla commented

Sorry, I wasn't clear. My preference would be there is a setting just for allowing contextual link to editing the EventSeries from event instances. My users have permission to edit Event Series when absolutely necessary, but they are encouraged not to. I would prefer not to confuse them by giving them an extra link they may accidentally use.

megakeegman’s picture

megakeegman commented

I will leave a note about your preference in the related issue: https://www.drupal.org/project/recurring_events/issues/3575567

chrisla’s picture

chrisla commented

My bad. Thanks for moving it there.

plopesc made their first commit to this issue’s fork.

plopesc opened merge request !193

plopesc’s picture

plopesc
he/him
Primary language Spanish
commented
Status: Active » Fixed

Contextual links added to both 2.0.x and 3.0.x branches. Thank you!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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