Skip to content

Commit 4cef7ee

Browse files
peschusterpeschuster
committed
feat: Support properties on license
Signed-off-by: Peter Schuster <p.schuster@pilz.de>
1 parent 69ac857 commit 4cef7ee

19 files changed

Lines changed: 353 additions & 22 deletions

‎src/models/license.ts‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ import type { Sortable } from '../_helpers/sortable'
2121
import type { LicenseAcknowledgement } from '../enums/licenseAcknowledgement'
2222
import type { SpdxId } from '../spdx'
2323
import type { Attachment } from './attachment'
24+
import { PropertyRepository } from './property'
2425

2526
/**
2627
* (SPDX) License Expression.
@@ -64,11 +65,13 @@ class DisjunctiveLicenseBase {
6465
acknowledgement?: LicenseAcknowledgement
6566
text?: Attachment
6667
#url?: URL | string
68+
properties: PropertyRepository
6769

6870
constructor (op: OptionalDisjunctiveLicenseProperties = {}) {
6971
this.acknowledgement = op.acknowledgement
7072
this.text = op.text
7173
this.url = op.url
74+
this.properties = op.properties ?? new PropertyRepository()
7275
}
7376

7477
get url (): URL | string | undefined {
@@ -86,6 +89,7 @@ interface OptionalDisjunctiveLicenseProperties {
8689
acknowledgement?: DisjunctiveLicenseBase['acknowledgement']
8790
text?: DisjunctiveLicenseBase['text']
8891
url?: DisjunctiveLicenseBase['url']
92+
properties?: DisjunctiveLicenseBase['properties']
8993
}
9094

9195
export interface OptionalNamedLicenseProperties extends OptionalDisjunctiveLicenseProperties {}

‎src/serialize/json/normalize.ts‎

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -529,36 +529,44 @@ export class LicenseNormalizer extends BaseJsonNormalizer<Models.License> {
529529
}
530530

531531
#normalizeNamedLicense (data: Models.NamedLicense, options: NormalizerOptions): Normalized.NamedLicense {
532+
const spec = this._factory.spec
532533
const url = escapeUri(data.url?.toString())
533534
return {
534535
license: {
535536
name: data.name,
536-
acknowledgement: this._factory.spec.supportsLicenseAcknowledgement
537+
acknowledgement: spec.supportsLicenseAcknowledgement
537538
? data.acknowledgement
538539
: undefined,
539540
text: data.text === undefined
540541
? undefined
541542
: this._factory.makeForAttachment().normalize(data.text, options),
542543
url: JsonSchema.isIriReference(url)
543544
? url
545+
: undefined,
546+
properties: spec.supportsProperties(data) && spec.supportsLicenseProperties && data.properties.size > 0
547+
? this._factory.makeForProperty().normalizeIterable(data.properties, options)
544548
: undefined
545549
}
546550
}
547551
}
548552

549553
#normalizeSpdxLicense (data: Models.SpdxLicense, options: NormalizerOptions): Normalized.SpdxLicense {
554+
const spec = this._factory.spec
550555
const url = escapeUri(data.url?.toString())
551556
return {
552557
license: {
553558
id: data.id,
554-
acknowledgement: this._factory.spec.supportsLicenseAcknowledgement
559+
acknowledgement: spec.supportsLicenseAcknowledgement
555560
? data.acknowledgement
556561
: undefined,
557562
text: data.text === undefined
558563
? undefined
559564
: this._factory.makeForAttachment().normalize(data.text, options),
560565
url: JsonSchema.isIriReference(url)
561566
? url
567+
: undefined,
568+
properties: spec.supportsProperties(data) && spec.supportsLicenseProperties && data.properties.size > 0
569+
? this._factory.makeForProperty().normalizeIterable(data.properties, options)
562570
: undefined
563571
}
564572
}

‎src/serialize/json/types.ts‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -196,6 +196,7 @@ export namespace Normalized {
196196
acknowledgement?: Enums.LicenseAcknowledgement
197197
text?: Attachment
198198
url?: string
199+
properties?: Property[]
199200
}
200201
}
201202

@@ -206,6 +207,7 @@ export namespace Normalized {
206207
acknowledgement?: Enums.LicenseAcknowledgement
207208
text?: Attachment
208209
url?: string
210+
properties?: Property[]
209211
}
210212
}
211213

‎src/serialize/xml/normalize.ts‎

Lines changed: 24 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -699,12 +699,20 @@ export class LicenseNormalizer extends BaseXmlNormalizer<Models.License> {
699699
}
700700

701701
#normalizeNamedLicense (data: Models.NamedLicense, options: NormalizerOptions): SimpleXml.Element {
702-
const url = escapeUri(data.url?.toString())
702+
const spec = this._factory.spec
703+
const url = escapeUri(data.url?.toString())
704+
const properties: SimpleXml.Element | undefined = spec.supportsProperties(data) && spec.supportsLicenseProperties && data.properties.size > 0
705+
? {
706+
type: 'element',
707+
name: 'properties',
708+
children: this._factory.makeForProperty().normalizeIterable(data.properties, options, 'property')
709+
}
710+
: undefined
703711
return {
704712
type: 'element',
705713
name: 'license',
706714
attributes: {
707-
acknowledgement: this._factory.spec.supportsLicenseAcknowledgement
715+
acknowledgement: spec.supportsLicenseAcknowledgement
708716
? data.acknowledgement
709717
: undefined
710718
},
@@ -715,18 +723,27 @@ export class LicenseNormalizer extends BaseXmlNormalizer<Models.License> {
715723
: this._factory.makeForAttachment().normalize(data.text, options, 'text'),
716724
XmlSchema.isAnyURI(url)
717725
? makeTextElement(url, 'url')
718-
: undefined
726+
: undefined,
727+
properties
719728
].filter(isNotUndefined)
720729
}
721730
}
722731

723732
#normalizeSpdxLicense (data: Models.SpdxLicense, options: NormalizerOptions): SimpleXml.Element {
724-
const url = escapeUri(data.url?.toString())
733+
const spec = this._factory.spec
734+
const url = escapeUri(data.url?.toString())
735+
const properties: SimpleXml.Element | undefined = spec.supportsProperties(data) && spec.supportsLicenseProperties && data.properties.size > 0
736+
? {
737+
type: 'element',
738+
name: 'properties',
739+
children: this._factory.makeForProperty().normalizeIterable(data.properties, options, 'property')
740+
}
741+
: undefined
725742
return {
726743
type: 'element',
727744
name: 'license',
728745
attributes: {
729-
acknowledgement: this._factory.spec.supportsLicenseAcknowledgement
746+
acknowledgement: spec.supportsLicenseAcknowledgement
730747
? data.acknowledgement
731748
: undefined
732749
},
@@ -737,7 +754,8 @@ export class LicenseNormalizer extends BaseXmlNormalizer<Models.License> {
737754
: this._factory.makeForAttachment().normalize(data.text, options, 'text'),
738755
XmlSchema.isAnyURI(url)
739756
? makeTextElement(url, 'url')
740-
: undefined
757+
: undefined,
758+
properties
741759
].filter(isNotUndefined)
742760
}
743761
}

‎src/spec/_protocol.ts‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ export interface _SpecProtocol {
4747
supportsMetadataProperties: boolean
4848
supportsExternalReferenceHashes: boolean
4949
supportsLicenseAcknowledgement: boolean
50+
supportsLicenseProperties: boolean
5051
supportsServices: boolean
5152
supportsToolsComponentsServices: boolean
5253
}
@@ -77,6 +78,7 @@ export class _Spec implements _SpecProtocol {
7778
readonly #supportsMetadataProperties: boolean
7879
readonly #supportsExternalReferenceHashes: boolean
7980
readonly #supportsLicenseAcknowledgement: boolean
81+
readonly #supportsLicenseProperties: boolean
8082
readonly #supportsServices: boolean
8183
readonly #supportsToolsComponentsServices: boolean
8284

@@ -100,6 +102,7 @@ export class _Spec implements _SpecProtocol {
100102
supportsMetadataProperties: boolean,
101103
supportsExternalReferenceHashes: boolean,
102104
supportsLicenseAcknowledgement: boolean,
105+
supportsLicenseProperties: boolean,
103106
supportsServices: boolean,
104107
supportsToolsComponentsServices: boolean
105108
) {
@@ -121,6 +124,7 @@ export class _Spec implements _SpecProtocol {
121124
this.#supportsMetadataProperties = supportsMetadataProperties
122125
this.#supportsExternalReferenceHashes = supportsExternalReferenceHashes
123126
this.#supportsLicenseAcknowledgement = supportsLicenseAcknowledgement
127+
this.#supportsLicenseProperties = supportsLicenseProperties
124128
this.#supportsServices = supportsServices
125129
this.#supportsToolsComponentsServices = supportsToolsComponentsServices
126130
}
@@ -204,6 +208,10 @@ export class _Spec implements _SpecProtocol {
204208
return this.#supportsLicenseAcknowledgement
205209
}
206210

211+
get supportsLicenseProperties (): boolean {
212+
return this.#supportsLicenseProperties
213+
}
214+
207215
get supportsServices (): boolean {
208216
return this.#supportsServices
209217
}

‎src/spec/consts.ts‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,7 @@ export const Spec1dot2: Readonly<_SpecProtocol> = Object.freeze(new _Spec(
8888
false,
8989
false,
9090
false,
91+
false,
9192
true,
9293
false
9394
))
@@ -153,6 +154,7 @@ export const Spec1dot3: Readonly<_SpecProtocol> = Object.freeze(new _Spec(
153154
true,
154155
true,
155156
false,
157+
false,
156158
true,
157159
false
158160
))
@@ -225,6 +227,7 @@ export const Spec1dot4: Readonly<_SpecProtocol> = Object.freeze(new _Spec(
225227
true,
226228
true,
227229
false,
230+
false,
228231
true,
229232
false
230233
))
@@ -327,6 +330,7 @@ export const Spec1dot5: Readonly<_SpecProtocol> = Object.freeze(new _Spec(
327330
true,
328331
false,
329332
true,
333+
true,
330334
true
331335
))
332336

@@ -433,6 +437,7 @@ export const Spec1dot6: Readonly<_SpecProtocol> = Object.freeze(new _Spec(
433437
true,
434438
true,
435439
true,
440+
true,
436441
true
437442
))
438443

@@ -547,6 +552,7 @@ export const Spec1dot7: Readonly<_SpecProtocol> = Object.freeze(new _Spec(
547552
true,
548553
true,
549554
true,
555+
true,
550556
true
551557
))
552558

‎tests/_data/models.js‎

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,11 @@ function createComplexStructure () {
158158
contentType: 'text/plain',
159159
encoding: Enums.AttachmentEncoding.Base64
160160
}),
161-
url: 'https://localhost/license'
161+
url: 'https://localhost/license',
162+
properties: new Models.PropertyRepository([
163+
new Models.Property('a', 'b'),
164+
new Models.Property('primaryLicense', 'true')
165+
])
162166
}))
163167
component.licenses.add((function (license) {
164168
license.text = new Models.Attachment('TUlUIExpY2Vuc2UKLi4uClRIRSBTT0ZUV0FSRSBJUyBQUk9WSURFRCAiQVMgSVMiLi4u')
@@ -167,7 +171,12 @@ function createComplexStructure () {
167171
license.url = new URL('https://spdx.org/licenses/MIT.html')
168172
license.acknowledgement = Enums.LicenseAcknowledgement.Declared
169173
return license
170-
})(new Models.SpdxLicense('MIT')))
174+
})(new Models.SpdxLicense('MIT', {
175+
properties: new Models.PropertyRepository([
176+
new Models.Property('c', 'd'),
177+
new Models.Property('primaryLicense', 'false')
178+
])
179+
})))
171180
component.publisher = 'the publisher'
172181
component.purl = new PackageURL('npm', 'acme', 'dummy-component', '1337-beta', undefined, undefined)
173182
component.scope = Enums.ComponentScope.Required

‎tests/_data/normalizeResults/json_sortedLists_spec1.5.json‎

Lines changed: 22 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎tests/_data/normalizeResults/json_sortedLists_spec1.6.json‎

Lines changed: 22 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎tests/_data/normalizeResults/json_sortedLists_spec1.7.json‎

Lines changed: 22 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)