Even there is a call in views_handler_field_date the theme function does not exists.

I think it would be enough to just copy the function from an old version, f.e DRUPAL-5

CommentFileSizeAuthor
#4 608926-theme-views-nodate.patch862 byteslongwave

Comments

dawehner’s picture

dawehner
Primary language German
commented
Priority: Normal » Minor

Thats minor

dawehner’s picture

dawehner
Primary language German
commented
Category: bug » task

This is a minor task, sorry but noone will ever need this.

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Postponed

This just is a problem if you are on a port of views to perhaps drupal8 :) Afaik this will not happen on a normal page run.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Title: theme_views_nodata is missing » Remove unimplemented theme('views_nodate') from the date handler
Status: Postponed » Needs review
StatusFileSize
new 608926-theme-views-nodate.patch862 bytes

I just noticed the call to theme('views_nodate') in the date handler and wondered why the theme function doesn't exist. I have a case where it would be invoked: if the date field is in a left joined table, the value could be null if there is no matching row.

Having said that, the empty text setting is obviously more flexible, so perhaps this should be removed? If you actually implement theme_views_nodate() then the empty text setting is always overridden. The attached patch removes the theme() call and applies to both 6.x-2.x and 6.x-3.x.

merlinofchaos’s picture

merlinofchaos commented
Version: 6.x-3.x-dev » 7.x-3.x-dev
Status: Needs review » Patch (to be ported)

Applied to 6.x-2.x and 6.x-3.x

dawehner’s picture

dawehner
Primary language German
commented
Status: Patch (to be ported) » Fixed

Ported to 7.x-3.x

Status: Fixed » Closed (fixed)

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