Solr
  1. Solr
  2. SOLR-451

analysis.jsp should use POST instead of GET and it would be nice to have a bigger input box

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3
    • Component/s: web gui
    • Labels:
      None

      Description

      It is useful to input larger Strings for analysis into the Solr admin analysis page, but the GET limitation prevents this. Thus, analysis.jsp and analysis.xsl should be changed to use POST instead. It would also be nicer to have a bigger input for the textarea (8+ cols)

      Patch to follow.

      1. SOLR-451.patch
        2 kB
        Grant Ingersoll

        Activity

        Hide
        Yonik Seeley added a comment -

        Why the bigger input for the text area? Anything really big would just be cut-n-pasted, no?
        And the visible area doesn't affect how much text you can actually add (because of scrollbars), right?
        I agree with using POST to work around URL size limitations.

        Show
        Yonik Seeley added a comment - Why the bigger input for the text area? Anything really big would just be cut-n-pasted, no? And the visible area doesn't affect how much text you can actually add (because of scrollbars), right? I agree with using POST to work around URL size limitations.
        Hide
        Grant Ingersoll added a comment -

        Why the bigger input for the text area? Anything really big would just be cut-n-pasted, no?

        It's just easier to read once it is in there, instead of scrolling through lots and lots of lines or having to switch back and forth between browser and a text editor.

        Actually, I think it would be nicer if it were taller and wider.

        But, I am fine w/ leaving the size as long as the POST gets fixed.

        Show
        Grant Ingersoll added a comment - Why the bigger input for the text area? Anything really big would just be cut-n-pasted, no? It's just easier to read once it is in there, instead of scrolling through lots and lots of lines or having to switch back and forth between browser and a text editor. Actually, I think it would be nicer if it were taller and wider. But, I am fine w/ leaving the size as long as the POST gets fixed.
        Hide
        Grant Ingersoll added a comment -

        Committed revision 630573

        Show
        Grant Ingersoll added a comment - Committed revision 630573
        Hide
        Hoss Man added a comment -

        This bug was modified as part of a bulk update using the criteria...

        • Marked "Resolved" and "Fixed"
        • Had no "Fix Version" versions
        • Was listed in the CHANGES.txt for 1.3 as of today 2008-03-15

        The Fix Version for all 29 issues found was set to 1.3, email notification was suppressed to prevent excessive email.

        For a list of all the issues modified, search jira comments for this (hopefully) unique string: batch20070315hossman1

        Show
        Hoss Man added a comment - This bug was modified as part of a bulk update using the criteria... Marked "Resolved" and "Fixed" Had no "Fix Version" versions Was listed in the CHANGES.txt for 1.3 as of today 2008-03-15 The Fix Version for all 29 issues found was set to 1.3, email notification was suppressed to prevent excessive email. For a list of all the issues modified, search jira comments for this (hopefully) unique string: batch20070315hossman1

          People

          • Assignee:
            Unassigned
            Reporter:
            Grant Ingersoll
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development