Paths

Table of Contentst

Handle NULL value
ClosedPublic
Actions

Authored by dereckson on Feb 27 2022, 01:36.

Tags

None

Referenced Files

	F35262203: D2563.id6475.diff
	Wed, Jul 8, 12:58

	F35262183: D2563.id6474.diff
	Wed, Jul 8, 12:58

	F35259323: D2563.diff
	Wed, Jul 8, 11:39

	F35224681: D2563.id6474.diff
	Tue, Jul 7, 21:48

	Unknown Object (File)
	Fri, Jul 3, 13:37

	Unknown Object (File)
	Fri, Jul 3, 13:29

	Unknown Object (File)
	Thu, Jul 2, 09:50

	Unknown Object (File)
	Thu, Jul 2, 08:44

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

rINFRAWWWac3264504858: Handle NULL value

Summary

The dump didn't handle the NULL type.

This created an issue, as a NULL value in JavaScript:

is an object (sic)
doesn't have a constructor property in prototype

Test Plan

runas: {{ bot['runas'] | default('') }}
now renders as runas: NULL.

Diff Detail

Repository

rINFRAWWW Nasqueron infrastructure servers website

Lint

Lint Passed

Unit

No Test Coverage

Branch

main

Build Status

Buildable 4013
Build 4265: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Feb 27 2022, 01:36

dereckson created this revision.

Harbormaster completed remote builds in B4013: Diff 6474.Feb 27 2022, 01:36

dereckson accepted this revision.Feb 27 2022, 01:36

This revision is now accepted and ready to land.Feb 27 2022, 01:36

SCSS rules gives:

That should attract attention it's not an expected value.

Closed by commit rINFRAWWWac3264504858: Handle NULL value (authored by dereckson). · Explain WhyFeb 27 2022, 01:37

This revision was automatically updated to reflect the committed changes.

dereckson added a commit: rINFRAWWWac3264504858: Handle NULL value.

Revision Contents
Changeset List

Path

Size

src/

assets/

js/

salt-config.js

4 lines

scss/

components/

_salt-config.scss

6 lines

Diff 6474

View Options

src/assets/js/salt-config.js