The facetapi.js file is missing a number of semicolons at the end of functions. These can cause issues when JS files are aggregated and/or compressed.

Also, the file code is not indented as it should be following Drupal programming standards.

Please review the attached patch. Thanks.

CommentFileSizeAuthor
facetapi-js_formatting_issues-1.patch10.57 KBsgdev

Comments

ron_s created an issue. See original summary.

  • joseph.olstad committed 7f91e6e on 7.x-1.x authored by ron_s 
    Issue #3057285 by ron_s: Missing semicolons can cause aggregated and...
joseph.olstad’s picture

joseph.olstad
Primary language French
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.