Skip to content

Commit 5e50f76

Browse files
committed
Revert DevTools polish
It seems like we can't use constructor injection at the moment.
1 parent 8ded42c commit 5e50f76

File tree

2 files changed

+19
-32
lines changed

2 files changed

+19
-32
lines changed

spring-boot-project/spring-boot-devtools/src/main/java/org/springframework/boot/devtools/autoconfigure/RemoteDevToolsAutoConfiguration.java

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import org.apache.commons.logging.Log;
2424
import org.apache.commons.logging.LogFactory;
2525

26+
import org.springframework.beans.factory.annotation.Autowired;
2627
import org.springframework.boot.autoconfigure.EnableAutoConfiguration;
2728
import org.springframework.boot.autoconfigure.condition.ConditionalOnClass;
2829
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
@@ -107,15 +108,11 @@ public DispatcherFilter remoteDevToolsDispatcherFilter(AccessManager accessManag
107108
@ConditionalOnProperty(prefix = "spring.devtools.remote.restart", name = "enabled", matchIfMissing = true)
108109
static class RemoteRestartConfiguration {
109110

110-
private final DevToolsProperties properties;
111+
@Autowired
112+
private DevToolsProperties properties;
111113

112-
private final ServerProperties serverProperties;
113-
114-
RemoteRestartConfiguration(DevToolsProperties devToolsProperties,
115-
ServerProperties serverProperties) {
116-
this.properties = devToolsProperties;
117-
this.serverProperties = serverProperties;
118-
}
114+
@Autowired
115+
private ServerProperties serverProperties;
119116

120117
@Bean
121118
@ConditionalOnMissingBean

spring-boot-project/spring-boot-devtools/src/main/java/org/springframework/boot/devtools/remote/client/RemoteClientConfiguration.java

Lines changed: 14 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
import org.apache.commons.logging.LogFactory;
2929

3030
import org.springframework.beans.factory.InitializingBean;
31-
import org.springframework.beans.factory.ObjectProvider;
31+
import org.springframework.beans.factory.annotation.Autowired;
3232
import org.springframework.beans.factory.annotation.Value;
3333
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
3434
import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
@@ -75,12 +75,11 @@ public class RemoteClientConfiguration implements InitializingBean {
7575

7676
private final DevToolsProperties properties;
7777

78-
private final String remoteUrl;
78+
@Value("${remoteUrl}")
79+
private String remoteUrl;
7980

80-
public RemoteClientConfiguration(DevToolsProperties properties,
81-
@Value("${remoteUrl}") String remoteUrl) {
81+
public RemoteClientConfiguration(DevToolsProperties properties) {
8282
this.properties = properties;
83-
this.remoteUrl = remoteUrl;
8483
}
8584

8685
@Bean
@@ -135,25 +134,20 @@ private void logWarnings() {
135134
static class LiveReloadConfiguration
136135
implements ApplicationListener<ClassPathChangedEvent> {
137136

138-
private final DevToolsProperties properties;
137+
@Autowired
138+
private DevToolsProperties properties;
139139

140-
private final LiveReloadServer liveReloadServer;
140+
@Autowired(required = false)
141+
private LiveReloadServer liveReloadServer;
141142

142-
private final ClientHttpRequestFactory clientHttpRequestFactory;
143+
@Autowired
144+
private ClientHttpRequestFactory clientHttpRequestFactory;
143145

144146
@Value("${remoteUrl}")
145147
private String remoteUrl;
146148

147149
private ExecutorService executor = Executors.newSingleThreadExecutor();
148150

149-
LiveReloadConfiguration(DevToolsProperties properties,
150-
ObjectProvider<LiveReloadServer> liveReloadServer,
151-
ClientHttpRequestFactory clientHttpRequestFactory) {
152-
this.properties = properties;
153-
this.liveReloadServer = liveReloadServer.getIfAvailable();
154-
this.clientHttpRequestFactory = clientHttpRequestFactory;
155-
}
156-
157151
@Bean
158152
@RestartScope
159153
@ConditionalOnMissingBean
@@ -187,15 +181,11 @@ final ExecutorService getExecutor() {
187181
@ConditionalOnProperty(prefix = "spring.devtools.remote.restart", name = "enabled", matchIfMissing = true)
188182
static class RemoteRestartClientConfiguration {
189183

190-
private final DevToolsProperties properties;
191-
192-
private final String remoteUrl;
184+
@Autowired
185+
private DevToolsProperties properties;
193186

194-
RemoteRestartClientConfiguration(DevToolsProperties properties,
195-
@Value("${remoteUrl}") String remoteUrl) {
196-
this.properties = properties;
197-
this.remoteUrl = remoteUrl;
198-
}
187+
@Value("${remoteUrl}")
188+
private String remoteUrl;
199189

200190
@Bean
201191
public ClassPathFileSystemWatcher classPathFileSystemWatcher() {

0 commit comments

Comments
 (0)