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

Single vlan handling fix #127

Closed
wants to merge 2 commits into from
Closed

Conversation

po1vo
Copy link
Contributor

@po1vo po1vo commented Nov 22, 2023

glpi-injector fails with
Loading test.xml...ERROR: 500 Internal Server Error, array_is_list(): Argument #1 ($array) must be of type array, string given, called in /glpi/vendor/glpi-project/inventory_format/lib/php/Converter.php on line 1421
on the following data:

        <PORT>
          <VLANS>
            <VLAN>
              <NAME>default</NAME>
              <NUMBER>1</NUMBER>
            </VLAN>
          </VLANS>
        </PORT>

@trasher
Copy link
Collaborator

trasher commented Nov 22, 2023

Please provide an example file that can be added to conversion tests. There are already VLANS example in tests/data/6.xml file that looks like the same you give as example data.

@po1vo
Copy link
Contributor Author

po1vo commented Nov 22, 2023

Never mind, our xml file was corrupted and triggered the error.

@po1vo po1vo closed this Nov 22, 2023
@po1vo po1vo deleted the converter-bugfix branch November 22, 2023 06:52
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.

2 participants