Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.3
    • Component/s: modules/other
    • Labels:
      None
    • Lucene Fields:
      New, Patch Available

      Description

      While working with BooleanFilter I wanted a basic toString() for debugging.

      This is what I came up. It works ok for me.

      1. patch3.txt
        1 kB
        Jason Calabrese

        Activity

        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12562269 ] jira [ 12583274 ]
        Mark Thomas made changes -
        Workflow jira [ 12416884 ] Default workflow, editable Closed status [ 12562269 ]
        Michael Busch made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Mark Harwood made changes -
        Resolution Fixed [ 1 ]
        Lucene Fields [Patch Available, New] [New, Patch Available]
        Fix Version/s 2.3 [ 12312531 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Hide
        Mark Harwood added a comment -

        Committed in revision 595996

        Show
        Mark Harwood added a comment - Committed in revision 595996
        Jason Calabrese made changes -
        Attachment patch.txt [ 12369294 ]
        Jason Calabrese made changes -
        Attachment patch3.txt [ 12369306 ]
        Hide
        Jason Calabrese added a comment -

        Wrapped the whole toString with BooleanFilter to match the toString()'s for other filters

        Show
        Jason Calabrese added a comment - Wrapped the whole toString with BooleanFilter to match the toString()'s for other filters
        Jason Calabrese made changes -
        Attachment patch.txt [ 12369281 ]
        Jason Calabrese made changes -
        Attachment patch.txt [ 12369294 ]
        Hide
        Jason Calabrese added a comment -

        Oops, here's a corrected patch

        Show
        Jason Calabrese added a comment - Oops, here's a corrected patch
        Hide
        Erik Hatcher added a comment -

        Jason - the patch looks like it is generated backwards (minus signs, not plusses).

        Show
        Erik Hatcher added a comment - Jason - the patch looks like it is generated backwards (minus signs, not plusses).
        Jason Calabrese made changes -
        Field Original Value New Value
        Attachment patch.txt [ 12369281 ]
        Hide
        Jason Calabrese added a comment -

        Attached patch

        Show
        Jason Calabrese added a comment - Attached patch
        Jason Calabrese created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Jason Calabrese
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development