chore(package_info_plus): Update dependencies and do necessary changes

vbuberen · vbuberen · commit ba374c00e7b8 · 2026-04-09T17:52:14.000+03:00
diff --git a/packages/package_info_plus/package_info_plus/lib/src/file_attribute.dart b/packages/package_info_plus/package_info_plus/lib/src/file_attribute.dart
@@ -28,38 +28,38 @@ class FileAttributes {
   late final DateTime? lastWriteTime;
 
   FileAttributes(this.filePath) {
-    final (:creationTime, :lastWriteTime) =
-        getFileCreationAndLastWriteTime(filePath);
+    final (:creationTime, :lastWriteTime) = getFileCreationAndLastWriteTime(
+      filePath,
+    );
 
     this.creationTime = creationTime;
     this.lastWriteTime = lastWriteTime;
   }
 
-  static ({
-    DateTime? creationTime,
-    DateTime? lastWriteTime,
-  }) getFileCreationAndLastWriteTime(String filePath) {
+  static ({DateTime? creationTime, DateTime? lastWriteTime})
+  getFileCreationAndLastWriteTime(String filePath) {
     if (!File(filePath).existsSync()) {
       throw ArgumentError.value(filePath, 'filePath', 'File not present');
     }
 
-    final lptstrFilename = TEXT(filePath);
+    final lptstrFilename = filePath.toPcwstr();
     final lpFileInformation = calloc<FILEATTRIBUTEDATA>();
 
     try {
-      if (GetFileAttributesEx(lptstrFilename, 0, lpFileInformation) == 0) {
-        throw WindowsException(HRESULT_FROM_WIN32(GetLastError()));
+      final result = GetFileAttributesEx(
+        lptstrFilename,
+        GetFileExInfoStandard,
+        lpFileInformation,
+      );
+      if (!result.value) {
+        throw WindowsException(result.error.toHRESULT());
       }
 
       final FILEATTRIBUTEDATA fileInformation = lpFileInformation.ref;
 
       return (
-        creationTime: fileTimeToDartDateTime(
-          fileInformation.ftCreationTime,
-        ),
-        lastWriteTime: fileTimeToDartDateTime(
-          fileInformation.ftLastWriteTime,
-        ),
+        creationTime: fileTimeToDartDateTime(fileInformation.ftCreationTime),
+        lastWriteTime: fileTimeToDartDateTime(fileInformation.ftLastWriteTime),
       );
     } finally {
       free(lptstrFilename);
diff --git a/packages/package_info_plus/package_info_plus/lib/src/file_version_info.dart b/packages/package_info_plus/package_info_plus/lib/src/file_version_info.dart
@@ -65,12 +65,16 @@ class FileVersionInfo {
       for (final langCodepage in langCodepages) {
         final lang = toHex4(langCodepage[0]);
         final codepage = toHex4(langCodepage[1]);
-        final lpSubBlock = TEXT('\\StringFileInfo\\$lang$codepage\\$name');
-        final res =
-            VerQueryValue(_data.lpBlock, lpSubBlock, lplpBuffer.cast(), puLen);
+        final lpSubBlock = '\\StringFileInfo\\$lang$codepage\\$name'.toPcwstr();
+        final res = VerQueryValue(
+          _data.lpBlock,
+          lpSubBlock,
+          lplpBuffer.cast(),
+          puLen,
+        );
         free(lpSubBlock);
 
-        if (res != 0 && lplpBuffer.address != 0 && puLen.value > 0) {
+        if (res && lplpBuffer.address != 0 && puLen.value > 0) {
           return lplpBuffer.value.toDartString();
         }
       }
@@ -87,20 +91,22 @@ class FileVersionInfo {
       throw ArgumentError.value(filePath, 'filePath', 'File not present');
     }
 
-    final lptstrFilename = TEXT(filePath);
-    final dwLen = GetFileVersionInfoSize(lptstrFilename, nullptr);
+    final lptstrFilename = filePath.toPcwstr();
+    final sizeResult = GetFileVersionInfoSize(lptstrFilename, null);
+    final dwLen = sizeResult.value;
 
     final lpData = calloc<BYTE>(dwLen); // freed by the dispose() method
-    final lpSubBlock = TEXT(r'\VarFileInfo\Translation');
+    final lpSubBlock = r'\VarFileInfo\Translation'.toPcwstr();
     final lpTranslate = calloc<Pointer<LANGANDCODEPAGE>>();
     final puLen = calloc<UINT>();
     try {
-      if (GetFileVersionInfo(lptstrFilename, NULL, dwLen, lpData) == 0) {
-        throw WindowsException(HRESULT_FROM_WIN32(GetLastError()));
+      final infoResult = GetFileVersionInfo(lptstrFilename, dwLen, lpData);
+      if (!infoResult.value) {
+        throw WindowsException(infoResult.error.toHRESULT());
       }
 
-      if (VerQueryValue(lpData, lpSubBlock, lpTranslate.cast(), puLen) == 0) {
-        throw WindowsException(HRESULT_FROM_WIN32(GetLastError()));
+      if (!VerQueryValue(lpData, lpSubBlock, lpTranslate.cast(), puLen)) {
+        throw WindowsException(GetLastError().toHRESULT());
       }
       return FileVersionInfoData(lpBlock: lpData, lpLang: lpTranslate.value);
     } finally {
diff --git a/packages/package_info_plus/package_info_plus/lib/src/package_info_plus_linux.dart b/packages/package_info_plus/package_info_plus/lib/src/package_info_plus_linux.dart
@@ -45,7 +45,8 @@ class PackageInfoPlusLinuxPlugin extends PackageInfoPlatform {
   }
 
   Future<({DateTime? created, DateTime? modified})> _getExeAttributes(
-      String exePath) async {
+    String exePath,
+  ) async {
     try {
       final statResult = await Process.run(
         'stat',
@@ -58,8 +59,9 @@ class PackageInfoPlusLinuxPlugin extends PackageInfoPlatform {
         return await _fallbackAttributes(exePath);
       }
 
-      final String stdout =
-          statResult.stdout is String ? statResult.stdout : '';
+      final String stdout = statResult.stdout is String
+          ? statResult.stdout
+          : '';
 
       if (stdout.split(',').length != 2) {
         return await _fallbackAttributes(exePath);
@@ -74,24 +76,24 @@ class PackageInfoPlusLinuxPlugin extends PackageInfoPlatform {
       );
       final modificationTime = _parseSecondsString(modificationMillis);
 
-      return (
-        created: creationTime,
-        modified: modificationTime,
-      );
+      return (created: creationTime, modified: modificationTime);
     } catch (_) {
       return (created: null, modified: null);
     }
   }
 
   Future<({DateTime created, DateTime modified})> _fallbackAttributes(
-      String exePath) async {
+    String exePath,
+  ) async {
     final modifiedTime = await File(exePath).lastModified();
 
     return (created: modifiedTime, modified: modifiedTime);
   }
 
-  DateTime? _parseSecondsString(String? secondsString,
-      {bool allowZero = true}) {
+  DateTime? _parseSecondsString(
+    String? secondsString, {
+    bool allowZero = true,
+  }) {
     if (secondsString == null) {
       return null;
     }
diff --git a/packages/package_info_plus/package_info_plus/lib/src/package_info_plus_web.dart b/packages/package_info_plus/package_info_plus/lib/src/package_info_plus_web.dart
@@ -17,7 +17,7 @@ class PackageInfoPlusWebPlugin extends PackageInfoPlatform {
 
   /// Create plugin with http client and asset manager for testing purposes.
   PackageInfoPlusWebPlugin([this._client, AssetManager? assetManagerMock])
-      : _assetManager = assetManagerMock ?? assetManager;
+    : _assetManager = assetManagerMock ?? assetManager;
 
   /// Registers this class as the default instance of [PackageInfoPlatform].
   static void registerWith(Registrar registrar) {
@@ -48,18 +48,19 @@ class PackageInfoPlusWebPlugin extends PackageInfoPlatform {
 
     // Add file and cachebuster query
     return uri.replace(
-        query: 'cachebuster=$cacheBuster',
-        pathSegments: [...segments, 'version.json']);
+      query: 'cachebuster=$cacheBuster',
+      pathSegments: [...segments, 'version.json'],
+    );
   }
 
   @override
   Future<PackageInfoData> getAll({String? baseUrl}) async {
     final int cacheBuster = clock.now().millisecondsSinceEpoch;
     final Map<String, dynamic> versionMap =
         await _getVersionMap(baseUrl, cacheBuster) ??
-            await _getVersionMap(_assetManager.baseUrl, cacheBuster) ??
-            await _getVersionMap(web.window.document.baseURI, cacheBuster) ??
-            {};
+        await _getVersionMap(_assetManager.baseUrl, cacheBuster) ??
+        await _getVersionMap(web.window.document.baseURI, cacheBuster) ??
+        {};
 
     return PackageInfoData(
       appName: versionMap['app_name'] ?? '',
diff --git a/packages/package_info_plus/package_info_plus/pubspec.yaml b/packages/package_info_plus/package_info_plus/pubspec.yaml
@@ -28,25 +28,24 @@ flutter:
         dartPluginClass: PackageInfoPlusWindowsPlugin
 
 dependencies:
-  ffi: ^2.0.1
+  clock: ^1.1.2
+  ffi: ^2.2.0
   flutter:
     sdk: flutter
   flutter_web_plugins:
     sdk: flutter
-  http: ">=0.13.5 <2.0.0"
-  meta: ^1.8.0
-  path: ^1.8.2
+  http: ^1.6.0
+  meta: ^1.17.0
   package_info_plus_platform_interface: ^3.2.1
-  web: ">=0.5.0 <2.0.0"
-  win32: ">=5.5.3 <7.0.0"
-  clock: ^1.1.1
+  path: ^1.9.1
+  web: ^1.1.1
+  win32: ^6.0.0
 
 dev_dependencies:
-  flutter_lints: ">=4.0.0 <6.0.0"
+  flutter_lints: ^6.0.0
   flutter_test:
     sdk: flutter
-  test: ^1.22.0
 
 environment:
-  sdk: ">=3.3.0 <4.0.0"
-  flutter: ">=3.19.0"
+  sdk: ">=3.11.0 <4.0.0"
+  flutter: ">=3.41.0"
diff --git a/packages/package_info_plus/package_info_plus/test/package_info_plus_windows_test.dart b/packages/package_info_plus/package_info_plus/test/package_info_plus_windows_test.dart
@@ -62,11 +62,14 @@ void main() {
     const missingFile = 'C:\\macos\\system128\\colonel.dll';
 
     expect(
-        () => FileVersionInfo(missingFile),
-        throwsA(isArgumentError.having(
+      () => FileVersionInfo(missingFile),
+      throwsA(
+        isArgumentError.having(
           (e) => e.message,
           'message',
           startsWith('File not present'),
-        )));
+        ),
+      ),
+    );
   });
 }
diff --git a/packages/package_info_plus/package_info_plus/test/package_info_test.dart b/packages/package_info_plus/package_info_plus/test/package_info_test.dart
@@ -20,51 +20,48 @@ void main() {
   final mockUpdateTime = now;
 
   TestDefaultBinaryMessengerBinding.instance.defaultBinaryMessenger
-      .setMockMethodCallHandler(
-    channel,
-    (MethodCall methodCall) async {
-      log.add(methodCall);
-      switch (methodCall.method) {
-        case 'getAll':
-          return <String, dynamic>{
-            'appName': 'package_info_example',
-            'buildNumber': '1',
-            'packageName': 'io.flutter.plugins.packageinfoexample',
-            'version': '1.0',
-            'installerStore': null,
-            'installTime': mockInstallTime.millisecondsSinceEpoch.toString(),
-            'updateTime': mockUpdateTime.millisecondsSinceEpoch.toString(),
-          };
-        default:
-          assert(false);
-          return null;
-      }
-    },
-  );
+      .setMockMethodCallHandler(channel, (MethodCall methodCall) async {
+        log.add(methodCall);
+        switch (methodCall.method) {
+          case 'getAll':
+            return <String, dynamic>{
+              'appName': 'package_info_example',
+              'buildNumber': '1',
+              'packageName': 'io.flutter.plugins.packageinfoexample',
+              'version': '1.0',
+              'installerStore': null,
+              'installTime': mockInstallTime.millisecondsSinceEpoch.toString(),
+              'updateTime': mockUpdateTime.millisecondsSinceEpoch.toString(),
+            };
+          default:
+            assert(false);
+            return null;
+        }
+      });
 
   tearDown(() {
     log.clear();
   });
 
-  test('fromPlatform', () async {
-    final info = await PackageInfo.fromPlatform();
-    expect(info.appName, 'package_info_example');
-    expect(info.buildNumber, '1');
-    expect(info.packageName, 'io.flutter.plugins.packageinfoexample');
-    expect(info.version, '1.0');
-    expect(info.installerStore, null);
-    expect(info.installTime, mockInstallTime);
-    expect(info.updateTime, mockUpdateTime);
-    expect(
-      log,
-      <Matcher>[
-        isMethodCall('getAll', arguments: null),
-      ],
-    );
-  }, onPlatform: {
-    'linux':
-        const Skip('PackageInfoPlus on Linux does not use platform channels'),
-  });
+  test(
+    'fromPlatform',
+    () async {
+      final info = await PackageInfo.fromPlatform();
+      expect(info.appName, 'package_info_example');
+      expect(info.buildNumber, '1');
+      expect(info.packageName, 'io.flutter.plugins.packageinfoexample');
+      expect(info.version, '1.0');
+      expect(info.installerStore, null);
+      expect(info.installTime, mockInstallTime);
+      expect(info.updateTime, mockUpdateTime);
+      expect(log, <Matcher>[isMethodCall('getAll', arguments: null)]);
+    },
+    onPlatform: {
+      'linux': const Skip(
+        'PackageInfoPlus on Linux does not use platform channels',
+      ),
+    },
+  );
 
   test('Mock initial values', () async {
     PackageInfo.setMockInitialValues(
diff --git a/packages/package_info_plus/package_info_plus_platform_interface/pubspec.yaml b/packages/package_info_plus/package_info_plus_platform_interface/pubspec.yaml
@@ -7,15 +7,15 @@ repository: https://github.com/fluttercommunity/plus_plugins/tree/main/packages/
 dependencies:
   flutter:
     sdk: flutter
-  meta: ^1.8.0
-  plugin_platform_interface: ^2.1.4
+  meta: ^1.17.0
+  plugin_platform_interface: ^2.1.8
 
 dev_dependencies:
   flutter_test:
     sdk: flutter
-  mockito: ^5.4.0
-  flutter_lints: ">=4.0.0 <6.0.0"
+  mockito: ^5.6.4
+  flutter_lints: ^6.0.0
 
 environment:
-  sdk: ">=2.18.0 <4.0.0"
-  flutter: ">=3.3.0"
+  sdk: ">=3.11.0 <4.0.0"
+  flutter: ">=3.46.0"
