Bug #2664
closed
An "Rule modified" event log with a target change is not displayed correctly
Added by Jean VILVER almost 12 years ago.
Updated almost 12 years ago.
Category:
Web - Maintenance
Description
We should see a formated description of the change before the reason message.
Files
- Description updated (diff)
- Status changed from New to Pending technical review
- % Done changed from 0 to 100
- Status changed from Pending technical review to Discussion
- % Done changed from 100 to 90
Jean, I'm not sure about the modification in the file EventLogDetailsService. The serialization of a Rule has a "targets" node, and within this node there are "target" nodes; however, you are looking for rule \ "target" rather that rule \ "targets". Could you explain why ?
Thank you!
- Status changed from Discussion to Pending technical review
Nicolas CHARLES wrote:
Jean, I'm not sure about the modification in the file EventLogDetailsService. The serialization of a Rule has a "targets" node, and within this node there are "target" nodes; however, you are looking for rule \ "target" rather that rule \ "targets". Could you explain why ?
Thank you!
It's fixed in 60f53ca9471582e122ea2869fa7b955180eb9079.
- Status changed from Pending technical review to In progress
- % Done changed from 90 to 80
val l = group.serverList.toSeq
l match {
case Nil => Text("None")
won't work ! You can't properly match Nil with a Seq as Nil extends List[Nothing], and List extends LinearSeq, which extends Seq.
If you are unsure about why it shouldn't work, replace "Nil" by "EmptyCar", List[Nothing] by Car[Nothing], LinearSeq by VehiculeWith4Wheels, and Seq by Vehicule
The probability that the Vehicule your are matching the an EmptyCar is very little
- Status changed from In progress to Pending technical review
- % Done changed from 80 to 100
This is correct Jean, thank you
- Status changed from Pending technical review to Released
Also available in: Atom
PDF
Updated by Jean VILVER 
Updated by