Bug #4621

CronInfocom : problem of sending (date control error)

Added by yllen over 7 years ago. Updated over 7 years ago.

Status:ClosedStart date:10/25/2013
Priority:NormalDue date:
Assignee:yllen% Done:

100%

Category:Notifications / Alerts
Target version:0.84.3

Associated revisions

Revision 22104
Added by yllen over 7 years ago

[0.83] cronInfocom : problem of sending - see #4621

Revision 22105
Added by yllen over 7 years ago

[0.84] cronInfocom : problem of sending - see #4621

Revision 22106
Added by yllen over 7 years ago

cronInfocom : problem of sending - fixed #4621

Revision 22131
Added by moyo over 7 years ago

[0.83] CronInfocom : problem of sending (date control error) see #4621

Revision 22132
Added by moyo over 7 years ago

[0.84] CronInfocom : problem of sending (date control error) fixed #4621

Revision 22133
Added by moyo over 7 years ago

CronInfocom : problem of sending (date control error) see #4621

History

#1 Updated by moyo over 7 years ago

Dans le sujet du forum :
De plus, quand j'exécute l'action automatique infocom, elle ne trouve rien à faire bien qu'une garantie expire ce jour.

Normal la garantie à une date de fin à aujourd'hui donc est considérée comme toujours sous garantie ? la notification partira demain normalement.

#2 Updated by yllen over 7 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset r22106.

#3 Updated by moyo over 7 years ago

Désolé encore pas d'accord avec la solution proposée.
En plus tu compares une date avec un datetime. Ca fonctionne mais ne me semble pas vraiment très logique.

S'il y a une adaptation à faire c'est juste sur le <0 (même si je n'en suis pas persuadé)

#4 Updated by moyo over 7 years ago

  • Status changed from Resolved to Assigned
  • % Done changed from 100 to 50

#5 Updated by yllen over 7 years ago

Je me suis posé la question mais la comparaison de cette manière a également été faite https://forge.indepnet.net/projects/glpi/repository/revisions/20840, donc vu que cela marchait je pensais que la correction était correcte.

#6 Updated by moyo over 7 years ago

Ce n'est pas la même chose vu que tickets.date est un datetime et non une date simple

#7 Updated by moyo over 7 years ago

  • Status changed from Assigned to Resolved
  • % Done changed from 50 to 100

Applied in changeset r22132.

#8 Updated by moyo over 7 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF