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

ERROR: Fomod's requiredInstallFiles strips first character from moved folder content if source or target start with a slash #16617

Open
Idrinth opened this issue Oct 30, 2024 · 4 comments
Assignees
Labels

Comments

@Idrinth
Copy link

Idrinth commented Oct 30, 2024

Describe the bug
Functional

    <requiredInstallFiles>
        <folder source="required" destination=""/>
    </requiredInstallFiles>

Strips first character

    <requiredInstallFiles>
        <folder source="/required" destination="/"/>
    </requiredInstallFiles>

To Reproduce

  1. Install a fomod with the above listed content
  2. check files in generated output
  3. See behaviour similar to screenshot below

Expected behavior
Both example fomod parts above should result in the same structure, without random characters being stripped

Screenshots
vortex

Platform (please complete the following information):

  • OS:Windows 10
  • Vortex Version 1.9.5
@Idrinth
Copy link
Author

Idrinth commented Oct 30, 2024

As a sidenote, this behaviour is not the same as with any other file or folder path - there the stripping doesn't happen and it works as expected

@Idrinth
Copy link
Author

Idrinth commented Oct 30, 2024

Double checked MO2 - it behaves as expected above

@Idrinth Idrinth changed the title Fomod: requiredInstallFiles strips first character from moved folder content if source or target start with a slash ERROR: Fomod's requiredInstallFiles strips first character from moved folder content if source or target start with a slash Nov 3, 2024
@Idrinth
Copy link
Author

Idrinth commented Nov 4, 2024

@IDCs forgot to tag you, just noticed your status.

@IDCs IDCs added the bug 🐛 label Nov 5, 2024
@IDCs IDCs self-assigned this Nov 5, 2024
@IDCs
Copy link
Contributor

IDCs commented Nov 5, 2024

Thanks for the report - will get that fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants