Skip to content

fix(massiveaction): fix PDF export redirecting to item list#62

Merged
Rom1-B merged 3 commits intopluginsGLPI:mainfrom
MyvTsv:ticket43434
Apr 22, 2026
Merged

fix(massiveaction): fix PDF export redirecting to item list#62
Rom1-B merged 3 commits intopluginsGLPI:mainfrom
MyvTsv:ticket43434

Conversation

@MyvTsv
Copy link
Copy Markdown

@MyvTsv MyvTsv commented Apr 22, 2026

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !43434
  • Here is a brief description of what this PR does
    When using the “Export PDF” MassiveAction, the redirect to the PDF file may not occur. This fix removes the echo from the <script> tag and uses the setRedirect() method of the MassiveAction class to perform the redirect

Screenshots (if appropriate):

@MyvTsv MyvTsv requested review from Rom1-B and stonebuzz April 22, 2026 13:58
@MyvTsv MyvTsv self-assigned this Apr 22, 2026
@MyvTsv MyvTsv marked this pull request as ready for review April 22, 2026 14:24
Comment thread CHANGELOG.md Outdated
@Rom1-B Rom1-B merged commit e6f10ce into pluginsGLPI:main Apr 22, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants