Droids
  1. Droids
  2. DROIDS-111

Open the API of RegexURLFilter to other types of input

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.1.0
    • Fix Version/s: 0.1.0
    • Component/s: core
    • Labels:
      None

      Description

      The only type of input RegexURLFilter accepts is the name of a file (setFile), which is very restrictive.
      I'm suggesting to introduce another input type in the API - the regex expression, which makes much more sense than just being able to specify the regex expressions in an external file.

      1. DROIDS-111_v1.patch
        2 kB
        Eugen Paraschiv

        Issue Links

          Activity

          Eugen Paraschiv created issue -
          Eugen Paraschiv made changes -
          Field Original Value New Value
          Attachment DROIDS-111_v1.patch [ 12465858 ]
          Hide
          Richard Frovarp added a comment -

          Looks good to me.

          Show
          Richard Frovarp added a comment - Looks good to me.
          Richard Frovarp made changes -
          Link This issue duplicates DROIDS-77 [ DROIDS-77 ]
          Richard Frovarp made changes -
          Link This issue duplicates DROIDS-77 [ DROIDS-77 ]
          Richard Frovarp made changes -
          Link This issue is duplicated by DROIDS-77 [ DROIDS-77 ]
          Hide
          Eugen Paraschiv added a comment -

          Good, so what's the next step to get this committed?
          Thanks.

          Show
          Eugen Paraschiv added a comment - Good, so what's the next step to get this committed? Thanks.
          Hide
          Otis Gospodnetic added a comment - - edited

          Tabs should be spaces. Maybe a unit test would be good to add.
          Patch applies cleanly, existing unit tests still pass, committed - thanks!

          Now could somebody with JIRA privileges mark this issue as Fixed, please?

          Show
          Otis Gospodnetic added a comment - - edited Tabs should be spaces. Maybe a unit test would be good to add. Patch applies cleanly, existing unit tests still pass, committed - thanks! Now could somebody with JIRA privileges mark this issue as Fixed, please?
          Thorsten Scherler made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          17d 21h 10m 1 Thorsten Scherler 26/Dec/10 20:15

            People

            • Assignee:
              Unassigned
              Reporter:
              Eugen Paraschiv
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development