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

[469c] Import sound/music into group None in Sound/Music Browser, place it in random new group, instead of group None. #1345

Closed
SeriousBuggie opened this issue Aug 10, 2023 · 2 comments
Labels
bug This is a confirmed bug internal-fam UnrealEd This is an issue in Unreal Editor
Milestone

Comments

@SeriousBuggie
Copy link
Collaborator

Reproduce:

  1. Import KR-Rankin.zip into Group None of new/some package.
  2. Import this file again into same package.

Expected result: Sound reloaded.
Actual result: Sound imported in new random PackageN group:
img
img
Log: FactoryCreateBinary: Sound with SoundFactory (0 0 Z:\UT2K4\Music\KR-Rankin.ogg)

@SeriousBuggie
Copy link
Collaborator Author

Import sound file goes by call next exec command:

	void ImportFile( FString Filename )
	{
		guard(WDlgImportSound::ImportFile);
		if (Group.Len())
			GEditor->Exec(*FString::Printf(TEXT("AUDIO IMPORT FILE=\"%ls\" NAME=\"%ls\" PACKAGE=\"%ls\" GROUP=\"%ls\""),
				*(*paFilenames)(iCurrentFilename), *Name, *Package, *Group));
		else
			GEditor->Exec(*FString::Printf(TEXT("AUDIO IMPORT FILE=\"%ls\" NAME=\"%ls\" PACKAGE=\"%ls\""),
				*(*paFilenames)(iCurrentFilename), *Name, *Package));
		unguard;
	}

@stijn-volckaert stijn-volckaert added enhancement Feature request UnrealEd This is an issue in Unreal Editor labels Aug 23, 2023
@SeriousBuggie SeriousBuggie added bug This is a confirmed bug internal-pf and removed enhancement Feature request labels Oct 11, 2023
@SeriousBuggie
Copy link
Collaborator Author

This related to different thing - Group None not use as empty group name. OGG not matter here.

@SeriousBuggie SeriousBuggie changed the title [469c] Import same OGG sound into Sound Browser, place it in random new group, instead of replace. [469c] Import sound into group None in Sound Browser, place it in random new group, instead of group None. Oct 12, 2023
@SeriousBuggie SeriousBuggie changed the title [469c] Import sound into group None in Sound Browser, place it in random new group, instead of group None. [469c] Import sound/music into group None in Sound/Music Browser, place it in random new group, instead of group None. Oct 12, 2023
@stijn-volckaert stijn-volckaert added this to the 469d milestone Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This is a confirmed bug internal-fam UnrealEd This is an issue in Unreal Editor
Projects
None yet
Development

No branches or pull requests

2 participants