Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(twig): Update twig version to preserve spaces #1847

Merged

Conversation

GMishx
Copy link
Member

@GMishx GMishx commented Dec 8, 2020

Description

In twig v1.38.4, the spaces between consecutive prints {{ }} {{ }} gets lost.
This issue is however limited to PHP7.4 and it causes issues with reports.
Updated the version to v1.42.5 to fix it.

Check twigphp/Twig#3248

Changes

  1. Update twig v1.38.4 => v1.42.5
  2. Escape obligation name in Admin >> License Admin >> Select License as it can contain special characters like "<.
  3. Print the current folder name instead of root folder in Browse view.

How to test

With PHP7.3 and PHP7.4

  1. Check if the folder name appears correctly in Browse page.
  2. Generate SPDX tag:value report and check if it looks ok.

Closes #1846

In twig v1.38.4, the spaces between consecutive prints `{{ }} {{ }}`
gets lost.
This issue is however limited to PHP7.4 and it causes issues with reports.
Updated the version to v1.42.5 to fix it.

Signed-off-by: Gaurav Mishra <mishra.gaurav@siemens.com>
Copy link
Contributor

@shaheemazmalmmd shaheemazmalmmd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code looks good. needs test

@shaheemazmalmmd shaheemazmalmmd merged commit e0b6d7d into fossology:master Dec 9, 2020
@GMishx GMishx deleted the fix/twig/fix-missing-spaces branch December 10, 2020 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SPDX tag:value report generation broken in Ubuntu 20.04 focal fossa
2 participants