Replace deprecated File.toURL()

[elsazac]

java.io.File.toURL() is deprecated and marked for removal. Replace it with file.toURI().toURL(), where toURL() is called on a java.net.URI instance, which is not deprecated and handles encoding correctly.

[github-actions[bot]]

Test Results

 1 824 files  ±0   1 824 suites  ±0   1h 39m 22s ⏱️ +2s  7 918 tests ±0   7 690 ✅ ±0  228 💤 ±0  0 ❌ ±0  23 841 runs  ±0  23 093 ✅ ±0  748 💤 ±0  0 ❌ ±0 

Results for commit 20d12195. ± Comparison against base commit 70589acc.

:recycle: This comment has been updated with latest results.

[eclipse-platform-bot]

This pull request changes some projects for the first time in this development cycle. Therefore the following files need a version increment:

bundles/org.eclipse.e4.ui.css.core/META-INF/MANIFEST.MF

An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch.

Git patch

From ff02de510f6048124e4233631559d25af42a4a52 Mon Sep 17 00:00:00 2001
From: Eclipse Platform Bot <[email protected]>
Date: Wed, 30 Apr 2025 07:05:56 +0000
Subject: [PATCH] Version bump(s) for 4.36 stream


diff --git a/bundles/org.eclipse.e4.ui.css.core/META-INF/MANIFEST.MF b/bundles/org.eclipse.e4.ui.css.core/META-INF/MANIFEST.MF
index 164a33caaa..459c82913a 100644
--- a/bundles/org.eclipse.e4.ui.css.core/META-INF/MANIFEST.MF
+++ b/bundles/org.eclipse.e4.ui.css.core/META-INF/MANIFEST.MF
@@ -4,7 +4,7 @@ Bundle-SymbolicName: org.eclipse.e4.ui.css.core;singleton:=true
 Bundle-Name: %pluginName
 Bundle-Vendor: %providerName
 Bundle-Localization: plugin
-Bundle-Version: 0.14.500.qualifier
+Bundle-Version: 0.14.600.qualifier
 Export-Package: org.eclipse.e4.ui.css.core;x-internal:=true,
  org.eclipse.e4.ui.css.core.css2;x-friends:="org.eclipse.e4.ui.css.swt.theme,org.eclipse.e4.ui.css.swt,org.eclipse.e4.ui.css.jface",
  org.eclipse.e4.ui.css.core.dom;x-friends:="org.eclipse.e4.ui.css.swt,org.eclipse.ui.views.properties.tabbed,org.eclipse.ui.forms",
-- 
2.49.0

Further information are available in Common Build Issues - Missing version increments.

[HannesWell]

Thank you @elsazac for the update. Unfortunately I misunderstood your previous comment. I meant that the method getPersistenceUrl() should just return a Path. To do this the Path representation of the baseUrl should be first extracted by using the method I'm introducing with

• https://github.com/eclipse-equinox/equinox/pull/691

but sadly that will probably not land anymore in this release cycle. But I plan to have this ready early next cycle and I suggest to put this on hold for that time. I'll keep you posted once that is ready, but also invite you to monitor that PR. That being said, once you got the Path of the baseUrl, then all checks and concatenations can just be done on the Path object and the same applies for the caller.

[elsazac]

but sadly that will probably not land anymore in this release cycle. But I plan to have this ready early next cycle and I suggest to put this on hold for that time. I'll keep you posted once that is ready, but also invite you to monitor that PR. That being said, once you got the Path of the baseUrl, then all checks and concatenations can just be done on the Path object and the same applies for the caller.

Sure, I’ll keep an eye on the PR and wait for your update. Once it’s in, I can revisit and refactor accordingly.