This happens on a month views when mini calendar is selected + week numbers are being shown.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geertvd created an issue. See original summary.

geertvd’s picture

Status: Active » Needs review
FileSize
1.96 KB

This gives us a link to a week view of the clicked week. The week view is still throwing an error, but that's a separate issue.
Right now I'm using Url::fromUri('base:' . $path .... which should be avoided, for that purpose I think we should rewrite calendar_granularity_path() to return a route rather then a path, I created a follow-up issue for that #2610940: calendar_granularity_path() should be rewritten to return a URL Object rather then a path

geertvd’s picture

The week view is still throwing an error, but that's a separate issue.

Created another follow-up for that #2610942: Fatal error thrown when adding a filter value in week view/day view

geertvd’s picture

I also did some research on $options = ['query' => !empty($this->styleInfo->append) ? $this->styleInfo->append : ''];. This seems to be legacy code as it's unused in 7.x also. Let's just remove that so.

geertvd’s picture

Assigned: geertvd » Unassigned

  • pjonckiere committed 728d4ca on 8.x-1.x authored by geertvd
    Issue #2610934 by geertvd: Call to undefined function Drupal\calendar\...
Anonymous’s picture

Status: Needs review » Fixed

Agreed that we shouldn't use Url::fromUri(), but this is good enough while refactoring. A follow-up for that is here: #2610940: calendar_granularity_path() should be rewritten to return a URL Object rather then a path.

Committed.

Status: Fixed » Closed (fixed)

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