Official CC-BY-4.0 license text is not being matched correctly by LicenseCompareHelper.matchingStandardLicenseIdsWithinText()
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 71
- Forks
- 44
- Ø Merge
- 12 Std. 54 Min.
- Gemergte PRs (30 T.)
- 7
Beschreibung
When org.spdx.utility.compare.LicenseCompareHelper.matchingStandardLicenseExceptionIdsWithinText() is run on the following CC-BY-*-4.0 license texts, it fails to find any matches:
Furthermore, org.spdx.utility.compare.LicenseCompareHelper.isStandardLicenseWithinText() is also returning false in each of these cases, however org.spdx.utility.compare.LicenseCompareHelper.isTextStandardLicense() (correctly) doesn't find any differences when run on the same texts (and when provided with the appropriate CC-BY-*-4.0 org.spdx.library.model.v3_0_1.expandedlicensing.ListedLicense object).
Perhaps we should add one or more CC-BY- cases to the compare consistency unit test?
This was reproduced with Spdx-Java-Library v2.0.0 and SPDX license list v3.26.0.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Abweichung in LicenseCompareHelper.matchingStandardLicenseIdsWithinText(), isStandardLicenseWithinText() und isTextStandardLicense() mit den aufgeführten CC-BY-*-4.0-Texten und den entsprechenden ListedLicense-Objekten zu reproduzieren. Sieh dir src/test/java/org/spdx/utility/compare/LicenseCompareHelperTest.java in der Nähe der Zeilen 986-994 an und füge eine Abdeckung für den Fall der Vergleichskonsistenz hinzu. Als erledigt gilt die Aufgabe, wenn die CC-BY-Varianten erkannt werden und die Konsistenzprüfungen bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100