Skip to content

Fix crashes in network and software PDF exports - #85

Open
angelmq152 wants to merge 1 commit into
pluginsGLPI:mainfrom
angelmq152:fix-export-crashes
Open

Fix crashes in network and software PDF exports#85
angelmq152 wants to merge 1 commit into
pluginsGLPI:mainfrom
angelmq152:fix-export-crashes

Conversation

@angelmq152

Copy link
Copy Markdown

Description

Fixes several independent GLPI 11 export failures:

  • handles network ports with no instantiation type instead of calling an invalid callback;
  • exports every IP address attached to a port, avoiding TooManyResultsException on dual-stack ports;
  • skips an absent installation user instead of passing null to formatUserName();
  • uses getEntitiesRestrictCriteria() with the GLPI array query builder when exporting software versions.

The upstream repository has issues disabled, so there is no issue number to reference.

Validation

  • git diff --check passed.
  • Local PHP/GLPI tests could not be run because this environment does not provide PHP; the project CI should validate against GLPI 11.

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.

1 participant