replacement for date field with publish_up as source 'param'

ggppdk
Offline
Administrator

11 years 4 months ago #38697 by ggppdk

Replied by ggppdk on topic replacement for date field with publish_up as source 'param'

Hello,

a wrong variable name ...
In date.php of date field find 2 places with:

Code:

$_value = ($date == 1) ? $item->publish_up : $item->publish_down;

and replace with:

Code:

$_value = ($date_source == 1) ? $item->publish_up : $item->publish_down;

i will commit fix ...
thanks for report

-- Flexicontent is Free but involves a big effort on our part.
Like the our support? (for a bug-free FC, despite having a long list of functions) Like the features? Like the ongoing development and future commitment to FLEXIcontent?
-- Add your voice to the FLEXIcontent JED listing with a 5-star...

Please Log in or Create an account to join the conversation.

rottenberg
Topic Author
Offline
Platinum Member

11 years 3 months ago #38731 by rottenberg

Replied by rottenberg on topic replacement for date field with publish_up as source 'param'

thank you very much.

Now it's working

Just a detail when you fix it : the label of the field remains wrong : The text shows end's publication date and not beginning's publication date
best regards

Michel

Please Log in or Create an account to join the conversation.

ggppdk
Offline
Administrator

11 years 3 months ago #38732 by ggppdk

Replied by ggppdk on topic replacement for date field with publish_up as source 'param'

Hello,

rottenberg wrote: Just a detail when you fix it : the label of the field remains wrong : The text shows end's publication date and not beginning's publication date

mmm , not a detail,
it is a same bug ... the wrong variable is used in 3 more places

Please, replace 3 occurrences of:

Code:

$date == 1

with:

Code:

$date_source == 1

thanks for pointing out,
i will commit this fix too
(if you have field in search index, then please recalculate them after applying the fix)
Regards

Please Log in or Create an account to join the conversation.