[Bug 61761] New: Conditional formatting rule evaluation doesn't like comparing cells of different types

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 61761] New: Conditional formatting rule evaluation doesn't like comparing cells of different types

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61761

            Bug ID: 61761
           Summary: Conditional formatting rule evaluation doesn't like
                    comparing cells of different types
           Product: POI
           Version: 3.17-FINAL
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: SS Common
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

There are a few wrinkles in evaluation unearthed by some Vaadin-Spreadsheet
unit tests.

The root is that the EvaluationConditonalFormatRule.OperatorEnum.isValid()
method takes <C extends Comparable<C>> inputs, and the calling code wants all
the inputs to be the same type to satisfy this.  However, when the comparison
operator is something like "not equals" then the result should be TRUE even
when the one value is a Number and the other is a String.  Currently it is
FALSE in this case, which doesn't match Excel's behavior, and is logically
incorrect (I'm not willing to say those two are always equivalent).

Also affects 4.0-dev, which is where I'll address it.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61761] Conditional formatting rule evaluation doesn't like comparing cells of different types

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61761

Greg Woolsey <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #1 from Greg Woolsey <[hidden email]> ---
Fixed in r1815298

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]