Uploaded image for project: 'Shindig'
  1. Shindig
  2. SHINDIG-1559

Trailer comma in features/opensearch/opensearch.js

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.0.3
    • Fix Version/s: 2.0.3
    • Component/s: Javascript
    • Labels:

      Description

      trailing comma in the js file that would cause an error in IE

      1. osclosure.txt
        0.6 kB
        Igor Belakovskiy
      2. osclosure.txt
        0.6 kB
        Igor Belakovskiy

        Activity

        Hide
        ibelakov Igor Belakovskiy added a comment -

        patch for the fix.

        Show
        ibelakov Igor Belakovskiy added a comment - patch for the fix.
        Hide
        woodstae Eric Woods added a comment -

        Igor - why comment the comma out and not simply remove it? Seems we'd want to remove unused or unnecessary code to keep things clean.

        Show
        woodstae Eric Woods added a comment - Igor - why comment the comma out and not simply remove it? Seems we'd want to remove unused or unnecessary code to keep things clean.
        Hide
        ibelakov Igor Belakovskiy added a comment -

        good point, adding updated patch.

        Show
        ibelakov Igor Belakovskiy added a comment - good point, adding updated patch.
        Hide
        ibelakov Igor Belakovskiy added a comment -

        Updated patch with Eric's suggestion

        Show
        ibelakov Igor Belakovskiy added a comment - Updated patch with Eric's suggestion
        Hide
        woodstae Eric Woods added a comment -

        Nevermind. There is a unit test below that is uncommented for testing, making the comma necessary. Patch applied, thanks Igor.

        Show
        woodstae Eric Woods added a comment - Nevermind. There is a unit test below that is uncommented for testing, making the comma necessary. Patch applied, thanks Igor.

          People

          • Assignee:
            Unassigned
            Reporter:
            ibelakov Igor Belakovskiy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development