Page MenuHomePhabricator

Let JSHint know that the `show_details` function is an exported global
ClosedPublic

Authored by joshuaspence on Dec 30 2014, 10:01 AM.
Tags
None
Referenced Files
F19812365: D11063.diff
Wed, Mar 4, 4:43 AM
F19807078: D11063.id26576.diff
Tue, Mar 3, 2:01 AM
F19651071: D11063.id26576.diff
Fri, Feb 6, 10:48 AM
F19636300: D11063.id26554.diff
Thu, Feb 5, 12:41 PM
F19545421: D11063.id26554.diff
Jan 24 2026, 6:03 AM
F19240147: D11063.diff
Dec 21 2025, 9:48 PM
F19209714: D11063.diff
Dec 17 2025, 12:15 AM
F18900679: D11063.id.diff
Nov 7 2025, 9:15 PM
Subscribers

Details

Summary

The show_details function is used by DarkConsole. Adding this comment silences the following JSHint warning:

>>> Lint for webroot/rsrc/js/core/behavior-error-log.js:

   Warning  (W098) JSHintW098
    'show_details' is defined but never used.

               5
               6 var current_details = null;
               7
    >>>        8 function show_details(row) {
               9   var node = JX.$('row-details-' + row);
              10
              11   if (current_details !== null) {
Test Plan

arc lint

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3356
Build 3363: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Let JSHint know that the `show_details` function is an exported global.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.

This should probably get cleaned up more completely at some point.

This revision is now accepted and ready to land.Dec 30 2014, 10:51 AM
This revision was automatically updated to reflect the committed changes.