Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-7588

naturalSort.js is provided as coffeescript instead of plain javascript

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 5.2, 6.0
    • 5.2.1, 5.3, 6.0
    • Admin UI
    • None
    • Fedora 21

      openjdk version "1.8.0_45"
      OpenJDK Runtime Environment (build 1.8.0_45-b14)
      OpenJDK 64-Bit Server VM (build 25.45-b02, mixed mode)

    Description

      The Dataimport tab of a core will hang with a loading screen or display the previously accessed tab instead of showing the expected dataimport screen.

      The console in Chrome has the following error log, but it's obvious to me that it's trying to run un-transpiled coffeescript:

      naturalSort.js?_=6.0.0:30 Uncaught SyntaxError: Unexpected token ILLEGAL
      jquery.sammy.js?_=6.0.0:120 [Fri May 22 2015 23:36:59 GMT-0700 (MST)] runRoute get #/db/dataimport
      dataimport.js?_=6.0.0:48 Uncaught ReferenceError: naturalSort is not defined
      

      The file in question can be viewed here: https://svn.apache.org/viewvc/lucene/dev/trunk/solr/webapp/web/js/lib/naturalSort.js?view=markup

      I was able to verify this in my own build as well as the nightly builds hosted on the Apache Jenkins server with the default DIH example (bin/solr start -e dih).

      After replacing the coffeescript file with one transpiled to javascript (available at https://github.com/jarinudom/naturalSort.js/blob/master/dist/naturalSort.js), the dataimport tab worked as expected.

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            elyograg Shawn Heisey
            dwo Derek Wood
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment