[jira] [Comment Edited] (UNOMI-624) avoid multiple logs about setPropertyValue: 'now'

["Kevan Jahanshahi (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/UNOMI-624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17574279#comment-17574279 ] 

Kevan Jahanshahi edited comment on UNOMI-624 at 8/2/22 3:18 PM:
----------------------------------------------------------------

Code using 'now' have been updated and also a temporary solution have been added to avoid logging the deprecated message.

(the log is only possible once every 6 hours, it's completely custom and should be removed by UNOMI-630 once we have a more global and suitable solution to this.)

PR: https://github.com/Jahia/jexperience/pull/639


was (Author: jkevan):
Code using 'now' have been updated and also a temporary solution have been added to avoid logging the deprecated message.

(the log is only possible once every 6 hours, it's completely custom and should be removed by UNOMI-630 once we have a more global and suitable solution to this.)

> avoid multiple logs about setPropertyValue: 'now' 
> --------------------------------------------------
>
>                 Key: UNOMI-624
>                 URL: https://issues.apache.org/jira/browse/UNOMI-624
>             Project: Apache Unomi
>          Issue Type: Improvement
>            Reporter: David Griffon
>            Assignee: Kevan Jahanshahi
>            Priority: Major
>             Fix For: 2.0.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> this log
> {code}
> Jul  7 07:59:04 jcustomer docker[10244]: 2022-07-07T07:59:04,723 | WARN  | qtp1412973081-331 | SetPropertyAction                | 226 - org.apache.unomi.plugins-base - 2.0.0.SNAPSHOT |  SetPropertyAction with setPropertyValue: 'now' is deprecated, please use 'setPropertyValueCurrentEventTimestamp' or 'setPropertyValueCurrentDate' instead of 'setPropertyValue' 
> {code}
> is happening a lot as soon as a goal as been set as it creates a rule with such action.
> as we don't migrate such rules. we need to warn the user only once. 
> We need also to update the code to use not deprecated stuff 
> https://github.com/apache/unomi/blob/130c29d391faac5b7d4bf49f07e9389b7a2c7eba/services/src/main/java/org/apache/unomi/services/impl/goals/GoalsServiceImpl.java#L173



--
This message was sent by Atlassian Jira
(v8.20.10#820010)