Support from Acquia helps fund testing for Drupal Acquia logo

Comments

minhtranite created an issue. See original summary.

Delphine Lepers’s picture

Apply patch naming convention, fix cs

Delphine Lepers’s picture

Delphine Lepers’s picture

jonnyeom’s picture

The patch seems to fail for Entity Link fields due to incompatibility

Here is the error.

Call to undefined method Drupal\views\Plugin\views\field\EntityLink::getItems() in Drupal\fullcalendar\Plugin\views\style\FullCalendar->prepareEvents() (line 296 of modules/contrib/fullcalendar/src/Plugin/views/style/FullCalendar.php).

jonnyeom’s picture

Although this does not resolve the incompatibility issue mentioned in #5, I updated the patch to resolve two other things.

  1. Undefined offset error on line 367 of fullcalendar.php
  2. The Custom URL was not being used.
jonnyeom’s picture

Accidentally added code from another patch.
Here is the patch with that extra code removed.

NickDickinsonWilde’s picture

cosolom’s picture

FileSize
4.39 KB

Rerolled

jrb’s picture

The patch in #9 does things differently from that in #8 and doesn't actually fix the title issue. The attached patch here does basically the same things as #8.

cosolom’s picture

@jrb, maybe you right. my patch is more relevant to problem when title and url for calendar not come from basic entity of view, but from entity added by relationship.

SocialNicheGuru’s picture

This conflict with the patch in https://www.drupal.org/project/fullcalendar/issues/3023612 that integrates date_recur module.

SocialNicheGuru’s picture

Status: Needs review » Needs work
ryankavalsky’s picture

I've just updated this patch to apply to 8.x-1.0-alpha3: Issue #3189593

jonnyeom’s picture

Status: Needs work » Reviewed & tested by the community

Just FYI,
The patch referenced in #3189593: Customized Field Titles and URLs Don't Work seems to work and latest 8.x-1.x branch