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

Add ARCropolis Compatability #36

Merged
merged 4 commits into from
Sep 11, 2023
Merged

Conversation

Starwarsfan2099
Copy link
Contributor

@Starwarsfan2099 Starwarsfan2099 commented Aug 29, 2023

This adds support for ARCropolis - a modding framework for Super Smash Bros. Ultimate. The majority of mods for Smash Ultimate on Gamebanana require ARCropolis and shouldn't be placed in /mods or /Atmposphere/contents. ARCropolis mods also don't follow the romfs and exefs structure. This pull request adds a setting option to install complete Smash mods to the ARCropolis mod location (/ultimate/mods). The setting will also be added to the settings.json file if it is not present so the config file doesn't have to be redistributed. And it defaults to off.

@PoloNX
Copy link
Owner

PoloNX commented Aug 30, 2023

Thanks for your pr ! I'll review and test it in few days :)

PoloNX

This comment was marked as outdated.

source/extract.cpp Outdated Show resolved Hide resolved
std::string nameWithoutExtension = "";
size_t extensionPos = archiveFile.find_last_of("."); // Find last dot position
if (extensionPos != std::string::npos && archiveFile.substr(extensionPos) == ".zip") {
nameWithoutExtension = archiveFile.substr(0, extensionPos); // Extract substring
Copy link
Owner

Choose a reason for hiding this comment

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

.7z extension must be supported

if (archive_entry_filetype(entry) == AE_IFDIR) {
// Create the directory
if (!std::filesystem::create_directory(outputPath)) {
Copy link
Owner

Choose a reason for hiding this comment

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

Good idea, maybe it would be interesting to do it even if ARCropolis is not used.

@PoloNX
Copy link
Owner

PoloNX commented Sep 4, 2023

I don't have smash bros ultimate so I can't test. Can you confirm me that it's working

@PoloNX
Copy link
Owner

PoloNX commented Sep 4, 2023

I have another idea. Like SimpleModManager can be launcher/downloader from The app maybe you can do the same for ARCropolis ?

@Starwarsfan2099
Copy link
Contributor Author

I have made the appropriate changes and retested, everything works as intended. Mods for ARCropolis and the standard game mods install fine. I also removed some unused code. As for installing ARCropolis from the HBAppStore, I'm not sure why it isn't provided as a package.

@PoloNX
Copy link
Owner

PoloNX commented Sep 11, 2023

Nice to see. The first commit has one mistake there is two time te condition with .zip. But you corrected it. I'll merge this very good pr

@PoloNX PoloNX merged commit dbae543 into PoloNX:master Sep 11, 2023
1 check 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.

2 participants