Skip to content

Commit 7d06025

Browse files
ahornaceVladimir Kotal
authored andcommitted
Remove deprecated attributes from configuration
1 parent 27c9740 commit 7d06025

File tree

1 file changed

+3
-59
lines changed

1 file changed

+3
-59
lines changed

opengrok-indexer/src/main/java/org/opengrok/indexer/configuration/Configuration.java

Lines changed: 3 additions & 59 deletions
Original file line numberDiff line numberDiff line change
@@ -148,15 +148,7 @@ public final class Configuration {
148148
*/
149149
private String includeRoot;
150150
private List<RepositoryInfo> repositories;
151-
/**
152-
* @deprecated This is kept around so not to break object deserialization
153-
* but it is ignored and cannot be truly set. This should mean that the
154-
* configuration is written leaving out this deprecated property; so after
155-
* some time it can be retired with the expectation that zero or a
156-
* miniscule number of production configurations still have this deprecated
157-
* property.
158-
*/
159-
private String urlPrefix;
151+
160152
private boolean generateHtml;
161153
/**
162154
* Default projects will be used, when no project is selected and no project
@@ -187,17 +179,7 @@ public final class Configuration {
187179
private String webappLAF;
188180
private RemoteSCM remoteScmSupported;
189181
private boolean optimizeDatabase;
190-
/**
191-
* @deprecated This is kept around so not to break object de-serialization.
192-
* <p>Anyone who is using `--lock on` will now be setting
193-
* {@link #luceneLocking} and resetting this field back to its default
194-
* value. This should mean that the configuration is written leaving out
195-
* this deprecated property; so after some time it can be retired with the
196-
* expectation that zero or a miniscule number of production configurations
197-
* still have this deprecated property.
198-
*/
199-
@Deprecated
200-
private boolean usingLuceneLocking;
182+
201183
private LuceneLockName luceneLocking = LuceneLockName.OFF;
202184
private boolean compressXref;
203185
private boolean indexVersionedFilesOnly;
@@ -477,12 +459,10 @@ public Configuration() {
477459
setStatisticsFilePath(null);
478460
//setTabSize(4);
479461
setTagsEnabled(false);
480-
//urlPrefix's constant value is moved to RuntimeEnvironment.
481462
//setUserPage("http://www.myserver.org/viewProfile.jspa?username=");
482463
// Set to empty string so we can append it to the URL
483464
// unconditionally later.
484465
setUserPageSuffix("");
485-
setUsingLuceneLocking(false);
486466
setVerbose(false);
487467
setWebappLAF("default");
488468
}
@@ -811,7 +791,7 @@ public void setDataRoot(String dataRoot) {
811791
}
812792

813793
/**
814-
* If {@link includeRoot} is not set, {@link dataRoot} will be returned.
794+
* If {@link #includeRoot} is not set, {@link #dataRoot} will be returned.
815795
* @return web include root directory
816796
*/
817797
public String getIncludeRoot() {
@@ -834,23 +814,6 @@ public void addRepositories(List<RepositoryInfo> repositories) {
834814
this.repositories.addAll(repositories);
835815
}
836816

837-
public String getUrlPrefix() {
838-
return urlPrefix;
839-
}
840-
841-
/**
842-
* Formerly this allowed setting the URL prefix to be used for the Java
843-
* class analyzer and for language cross-referencing (xref) when they
844-
* created HTML links. Now, a static value is used and transformed as
845-
* necessary to the servlet {@code contextPath} so that users can deploy
846-
* OpenGrok as they like.
847-
* @param urlPrefix ignored
848-
*/
849-
@Deprecated
850-
public void setUrlPrefix(String urlPrefix) {
851-
// ignore the value
852-
}
853-
854817
public void setGenerateHtml(boolean generateHtml) {
855818
this.generateHtml = generateHtml;
856819
}
@@ -1026,23 +989,6 @@ public void setOptimizeDatabase(boolean optimizeDatabase) {
1026989
this.optimizeDatabase = optimizeDatabase;
1027990
}
1028991

1029-
@Deprecated
1030-
public boolean isUsingLuceneLocking() {
1031-
return LuceneLockName.SIMPLE.equals(luceneLocking) ||
1032-
LuceneLockName.ON.equals(luceneLocking);
1033-
}
1034-
1035-
@Deprecated
1036-
public boolean getUsingLuceneLocking() {
1037-
return isUsingLuceneLocking();
1038-
}
1039-
1040-
@Deprecated
1041-
public void setUsingLuceneLocking(boolean useLuceneLocking) {
1042-
setLuceneLocking(useLuceneLocking ? LuceneLockName.ON :
1043-
LuceneLockName.OFF);
1044-
}
1045-
1046992
public LuceneLockName getLuceneLocking() {
1047993
return luceneLocking;
1048994
}
@@ -1053,8 +999,6 @@ public LuceneLockName getLuceneLocking() {
1053999
*/
10541000
public void setLuceneLocking(LuceneLockName value) {
10551001
this.luceneLocking = value;
1056-
// Set the following to default(boolean) regardless of `value'.
1057-
this.usingLuceneLocking = false;
10581002
}
10591003

10601004
public void setCompressXref(boolean compressXref) {

0 commit comments

Comments
 (0)