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

Preview 1.82.5 Release #618

Merged
merged 30 commits into from
Dec 11, 2024
Merged

Preview 1.82.5 Release #618

merged 30 commits into from
Dec 11, 2024

Conversation

bagusnl
Copy link
Member

@bagusnl bagusnl commented Dec 9, 2024

What's new? - 1.82.5

  • [Fix] Various weird errors that might be caused by Optimization preference, by @neon-nyan & @bagusnl
    • We are noticing quite odd behaviors with some functions, critically Sophon and Http downloader and also SevenZipExtractor modules
    • This error does not seem to be reproducible in Debug environment, but recent development by @neon-nyan found that setting the optimization profile to Speed may cause compiler over optimization in certain areas.
    • This fix set the project's compiler optimization to Size instead (which is the default .NET behaviour).
    • If this fixes those errors that were problematic then good, if not then we have to rethink about those problematic modules...
  • [Fix] Exceptions in Carousel methods, by @bagusnl
    • Third time the charm 🤞
  • [Fix] DirectoryNotFound exception in HTTP module, by @bagusnl
  • [Imp] Only recreate shortcut once, by @bagusnl
    • Please, no more re-pinning my shortcut...
  • [Fix] App crash when finalizing update, by @bagusnl
  • [Fix] Prevent image load fails when waifu2x failes to load / nulled, by @shatyuka
  • [Fix] (Attempt to) Fix corruption when extracting packages via SevenZipExtractor, by @neon-nyan

Templates

Changelog Prefixes
  **[New]**
  **[Imp]**
  **[Fix]**
  **[Loc]**
  **[Doc]**

Cryotechnic and others added 26 commits December 8, 2024 09:42
Also disable Reflection completely
To prevent need to re pin all shortcuts
Splitting some codes into several methods:
- CreateShortcut() - Create shortcuts from desired path
- GetIconLocationPaths() - Get Icon Location Paths and some attributes like appProductName to be used as AUMID for Toast Notification feature
Due to unified Game Repair and Cache Update into a single base class, this causes the message from Game Repair to appear instead Cache Update ones
100% reviewed source file: 'en_US.json'
on 'es_419'.
100% reviewed source file: 'en_US.json'
on 'zh_CN'.
@@ -1140,7 +1140,21 @@
return false;
}

private bool IsDiskPartitionExist(string path) => !string.IsNullOrEmpty(path) && !string.IsNullOrEmpty(Path.GetPathRoot(path)) && new DriveInfo(Path.GetPathRoot(path) ?? string.Empty).IsReady;
private bool IsDiskPartitionExist(string path)

Check notice

Code scanning / QDNET

RoslynAnalyzers Mark members as static Note

Member 'IsDiskPartitionExist' does not access instance data and can be marked as static
@@ -1140,7 +1140,21 @@
return false;
}

private bool IsDiskPartitionExist(string path) => !string.IsNullOrEmpty(path) && !string.IsNullOrEmpty(Path.GetPathRoot(path)) && new DriveInfo(Path.GetPathRoot(path) ?? string.Empty).IsReady;
private bool IsDiskPartitionExist(string path)

Check notice

Code scanning / QDNET

Member can be made static (shared) (private accessibility) Note

Method 'IsDiskPartitionExist' can be made static
@@ -283,6 +285,45 @@
}
}

private static NotificationService? _currentToastNotificationService;
internal static NotificationService? CurrentToastNotificationService

Check warning

Code scanning / QDNET

Return type of a function can be made non-nullable Warning

Return type of 'CurrentToastNotificationService' can be made non-nullable
get
{
// If toast notification service field is null, then initialize
if (_currentToastNotificationService == null)

Check notice

Code scanning / QDNET

Invert 'if' statement to reduce nesting Note

Invert 'if' statement to reduce nesting
_currentToastNotificationService = new NotificationService(ILoggerHelper.GetILogger("ToastCOM"));

// Get string for AumId registration
if (!string.IsNullOrEmpty(appAumIdName))

Check notice

Code scanning / QDNET

Invert 'if' statement to reduce nesting Note

Invert 'if' statement to reduce nesting
WindowUtility.CurrentToastNotificationService?.CreateToastNotifier();
notifier.Show(notificationService);
}
catch { }

Check warning

Code scanning / QDNET

Empty general catch clause Warning

Empty general catch clause suppresses any errors
@@ -1,13 +1,17 @@
using CollapseLauncher.Helper;
using CollapseLauncher.Helper.Background;
using CollapseLauncher.Helper.Database;

Check warning

Code scanning / QDNET

Redundant using directive Warning

Using directive is not required by the code and can be safely removed
SpawnGreetingsToastNotification(categorySelected, regionSelected);
}

private void SpawnGreetingsToastNotification(string? gameName, string? regionName)

Check notice

Code scanning / QDNET

RoslynAnalyzers Mark members as static Note

Member 'SpawnGreetingsToastNotification' does not access instance data and can be marked as static
SpawnGreetingsToastNotification(categorySelected, regionSelected);
}

private void SpawnGreetingsToastNotification(string? gameName, string? regionName)

Check notice

Code scanning / QDNET

Member can be made static (shared) (private accessibility) Note

Method 'SpawnGreetingsToastNotification' can be made static

private void SpawnGreetingsToastNotification(string? gameName, string? regionName)
{
if (!string.IsNullOrEmpty(gameName)

Check notice

Code scanning / QDNET

Invert 'if' statement to reduce nesting Note

Invert 'if' statement to reduce nesting
@bagusnl bagusnl merged commit 626989f into preview Dec 11, 2024
4 of 5 checks passed
Copy link

sentry-io bot commented Dec 11, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ **System.IO.IOException: IO_SharingViolation_File, **************AppData\Roaming\Microsoft\Windows\Start Menu\Programs\C... Program.cs in void MainEntryPoint.TryCleanupFal... View Issue

Did you find this useful? React with a 👍 or 👎

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.

4 participants