fix(sqllab): missing zero values while copy-to-clipboard

[justinpark]

SUMMARY

Since JS returns false for zero values, following value existence statement data[i][key] skips the zero values.

superset/superset-frontend/src/utils/common.js

Lines 100 to 106 in 0bf4e56

	if (data[i][key]) {
	row[j] = data[i][key];
	} else {
	row[j] = data[i][parseFloat(key)];
	}
	}
	result += `${Object.values(row).join('\t')}\n`;

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

copy-to-clipboard-for-zeros.mov

TESTING INSTRUCTIONS

1. Go to Sql Lab
2. Run the following query

SELECT 'copy to clipboard',
        0 as zero,
        1 as one

3. "Copy to clipboard"
4. Paste to a note

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #21153 (ce68697) into master (0bf4e56) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master   #21153   +/-   ##
=======================================
  Coverage   66.16%   66.16%           
=======================================
  Files        1773     1773           
  Lines       67689    67689           
  Branches     7218     7218           
=======================================
  Hits        44786    44786           
  Misses      21059    21059           
  Partials     1844     1844           

Flag	Coverage Δ
javascript	52.12% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
superset-frontend/src/utils/common.js	90.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[stephenLYZ]

LGTM