Solr
  1. Solr
  2. SOLR-2814

Core names that contain a "-" fail in new Admin Gui

    Details

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

      Working with Solr 4 trunk

      Description

      If you have a core with a "-" in the name, any clicks on it in the new web GUI seem to be ignored. A core named "uspatentgrant" works fine, but a core named "us-patent-grant" isn't openable in the GUI. Nothing is logged in the solr output either. I will attach a screenshot.

      1. SOLR-2814-core-names.patch
        4 kB
        Stefan Matheis (steffkes)
      2. solr-admin.png
        110 kB
        Eric Pugh

        Activity

        Eric Pugh created issue -
        Hide
        Eric Pugh added a comment -

        screenshot showing admin gui

        Show
        Eric Pugh added a comment - screenshot showing admin gui
        Eric Pugh made changes -
        Field Original Value New Value
        Attachment solr-admin.png [ 12498061 ]
        Hide
        Martijn van Groningen added a comment -

        I also noticed this today. I didn't know that this was the problem. Now that I have renamed core I know it is.
        The core with a dash does exist in Solr but it isn't possible to interact with core via the new gui.

        Show
        Martijn van Groningen added a comment - I also noticed this today. I didn't know that this was the problem. Now that I have renamed core I know it is. The core with a dash does exist in Solr but it isn't possible to interact with core via the new gui.
        Hide
        Eric Pugh added a comment -

        Much better description of the behavior of the bug!

        Show
        Eric Pugh added a comment - Much better description of the behavior of the bug!
        Hide
        Stefan Matheis (steffkes) added a comment -

        Thanks for Reporting this Eric, Martijn - patch attached, based on rev 1179955

        currently, the regex matches only \w\d .. which doesn't include dashes :/

        Show
        Stefan Matheis (steffkes) added a comment - Thanks for Reporting this Eric, Martijn - patch attached, based on rev 1179955 currently, the regex matches only \w\d .. which doesn't include dashes :/
        Stefan Matheis (steffkes) made changes -
        Attachment SOLR-2814-core-names.patch [ 12498123 ]
        Hide
        Martijn van Groningen added a comment -

        Thanks Stefan!

        Show
        Martijn van Groningen added a comment - Thanks Stefan!
        Hide
        Stefan Matheis (steffkes) added a comment -

        Hey Guys, can we commit this fix, if it's working for all? Just to have clean WC for potential fixes? Ty, Stefan

        Show
        Stefan Matheis (steffkes) added a comment - Hey Guys, can we commit this fix, if it's working for all? Just to have clean WC for potential fixes? Ty, Stefan
        Ryan McKinley made changes -
        Assignee Ryan McKinley [ ryantxu ]
        Ryan Shurr committed 1182555 (1 file)
        Reviews: none

        SOLR-2814: fix admin UI for corenames with a dash

        Hide
        Ryan McKinley added a comment -

        added in r1182555

        Thanks Stefan!

        are there other outstanding admin UI updates?

        Show
        Ryan McKinley added a comment - added in r1182555 Thanks Stefan! are there other outstanding admin UI updates?
        Ryan McKinley made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 4.0 [ 12314992 ]
        Resolution Fixed [ 1 ]
        Hide
        Stefan Matheis (steffkes) added a comment -

        Ryan,

        are there other outstanding admin UI updates?

        i'm not sure if the last patch from SOLR-2667 was already committed - but i'll check it tomorrow and include an updated patch .. just for the case

        Stefan

        Show
        Stefan Matheis (steffkes) added a comment - Ryan, are there other outstanding admin UI updates? i'm not sure if the last patch from SOLR-2667 was already committed - but i'll check it tomorrow and include an updated patch .. just for the case Stefan
        Uwe Schindler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Ryan McKinley
            Reporter:
            Eric Pugh
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development