Skip to content

Commit 98b720e

Browse files
authored
Merge pull request #87 from thc202/address-project-usages
Address warns and release 0.13.1
2 parents d141fa1 + 7d6cf81 commit 98b720e

File tree

5 files changed

+37
-18
lines changed

5 files changed

+37
-18
lines changed

CHANGELOG.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@ All notable changes to this project will be documented in this file.
44
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
55
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
66

7-
## [Unreleased]
7+
## [0.13.1] - 2025-02-27
8+
### Fixed
9+
- Address deprecation warnings related to project usage at runtime.
810

911
## [0.13.0] - 2025-02-27
1012
### Changed
@@ -246,7 +248,7 @@ Provided by the plugin:
246248
with a `ZapVersions.xml` from an add-on.
247249

248250

249-
[Unreleased]: https://github.com/zaproxy/gradle-plugin-add-on/compare/v0.13.0...HEAD
251+
[0.13.1]: https://github.com/zaproxy/gradle-plugin-add-on/compare/v0.13.0...v0.13.1
250252
[0.13.0]: https://github.com/zaproxy/gradle-plugin-add-on/compare/v0.12.0...v0.13.0
251253
[0.12.0]: https://github.com/zaproxy/gradle-plugin-add-on/compare/v0.11.0...v0.12.0
252254
[0.11.0]: https://github.com/zaproxy/gradle-plugin-add-on/compare/v0.10.0...v0.11.0

build.gradle.kts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ plugins {
77
}
88

99
group = "org.zaproxy.gradle"
10-
version = "0.14.0-SNAPSHOT"
10+
version = "0.13.1"
1111

1212
dependencies {
1313
implementation("commons-codec:commons-codec:1.17.1")

src/main/java/org/zaproxy/gradle/addon/jh/tasks/JavaHelpIndexer.java

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@
4040
import org.gradle.api.file.ConfigurableFileCollection;
4141
import org.gradle.api.file.Directory;
4242
import org.gradle.api.file.DirectoryProperty;
43+
import org.gradle.api.file.FileSystemOperations;
4344
import org.gradle.api.file.FileTree;
4445
import org.gradle.api.file.RegularFileProperty;
4546
import org.gradle.api.model.ObjectFactory;
@@ -108,6 +109,11 @@ protected ExecOperations getExecOperations() {
108109
throw new UnsupportedOperationException();
109110
}
110111

112+
@Inject
113+
protected FileSystemOperations getFs() {
114+
throw new UnsupportedOperationException();
115+
}
116+
111117
@InputFile
112118
@PathSensitive(PathSensitivity.RELATIVE)
113119
public RegularFileProperty getHelpset() {
@@ -213,7 +219,8 @@ private File getWorkingDirectory() {
213219
}
214220

215221
private void prepareWorkingDirectory(File wd) {
216-
getProject().delete(wd);
222+
getFs().delete(spec -> spec.delete(wd));
223+
217224
try {
218225
Files.createDirectories(wd.toPath());
219226
} catch (IOException e) {

src/main/java/org/zaproxy/gradle/addon/misc/CopyAddOn.java

Lines changed: 15 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,11 @@
2626
import java.nio.file.Path;
2727
import java.util.regex.Pattern;
2828
import java.util.stream.Stream;
29+
import javax.inject.Inject;
2930
import org.gradle.api.Project;
31+
import org.gradle.api.file.ConfigurableFileTree;
32+
import org.gradle.api.file.FileSystemOperations;
33+
import org.gradle.api.model.ObjectFactory;
3034
import org.gradle.api.provider.Property;
3135
import org.gradle.api.tasks.Copy;
3236
import org.gradle.api.tasks.Input;
@@ -46,9 +50,11 @@ public class CopyAddOn extends Copy {
4650
private static final String DEFAULT_DIR_PATH = "../zaproxy/zap/src/main/dist/plugin/";
4751

4852
private final Property<String> addOnId;
53+
private final ObjectFactory objects;
4954

5055
public CopyAddOn() {
5156
Project project = getProject();
57+
objects = project.getObjects();
5258
addOnId = project.getObjects().property(String.class);
5359

5460
from(project.getTasks().named("jarZapAddOn"));
@@ -81,6 +87,11 @@ public CopyAddOn() {
8187
});
8288
}
8389

90+
@Inject
91+
protected FileSystemOperations getFs() {
92+
throw new UnsupportedOperationException();
93+
}
94+
8495
@Input
8596
public Property<String> getAddOnId() {
8697
return addOnId;
@@ -93,17 +104,10 @@ public void optionInto(String dir) {
93104

94105
@Override
95106
public void copy() {
96-
Project project = getProject();
97-
project.delete(
98-
project.fileTree(
99-
getDestinationDir(),
100-
tree -> {
101-
tree.include(
102-
getAddOnId().get()
103-
+ "-*."
104-
+ Constants.ADD_ON_FILE_EXTENSION);
105-
})
106-
.getFiles());
107+
ConfigurableFileTree tree = objects.fileTree().from(getDestinationDir());
108+
tree.include(getAddOnId().get() + "-*." + Constants.ADD_ON_FILE_EXTENSION);
109+
getFs().delete(spec -> spec.delete(tree.getFiles()));
110+
107111
super.copy();
108112
}
109113
}

src/main/java/org/zaproxy/gradle/addon/misc/DeployAddOn.java

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,11 @@
2222
import java.io.File;
2323
import java.util.ArrayList;
2424
import java.util.List;
25+
import javax.inject.Inject;
2526
import org.apache.commons.lang3.SystemUtils;
2627
import org.gradle.api.DefaultTask;
2728
import org.gradle.api.file.ConfigurableFileCollection;
29+
import org.gradle.api.file.FileSystemOperations;
2830
import org.gradle.api.file.RegularFileProperty;
2931
import org.gradle.api.model.ObjectFactory;
3032
import org.gradle.api.provider.Property;
@@ -68,6 +70,11 @@ public DeployAddOn() {
6870
deleteStale = objects.property(Boolean.class).value(true);
6971
}
7072

73+
@Inject
74+
protected FileSystemOperations getFs() {
75+
throw new UnsupportedOperationException();
76+
}
77+
7178
@Option(option = "zap-home-dir", description = "The file system path to the ZAP home.")
7279
public void optionHomeDir(String dir) {
7380
homeDir.set(getProject().file(dir));
@@ -116,11 +123,10 @@ public void deploy() {
116123
homeDir.get(),
117124
fileDetails.getRelativePath().toString()));
118125
});
119-
getProject().delete(filesToDelete);
126+
getFs().delete(spec -> spec.delete(filesToDelete));
120127
}
121128

122-
getProject()
123-
.copy(
129+
getFs().copy(
124130
copySpec -> {
125131
copySpec.from(addOn, copySpecAddOn -> copySpecAddOn.into(PLUGIN_DIR));
126132
copySpec.from(files);

0 commit comments

Comments
 (0)