Skip to content

Commit 5f50aad

Browse files
committed
chore: bump plugin-api to 4.1.13
1 parent b0f393c commit 5f50aad

15 files changed

+52
-66
lines changed

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
<parent>
66
<groupId>org.ligoj.api</groupId>
77
<artifactId>plugin-parent</artifactId>
8-
<version>4.1.12</version>
8+
<version>4.1.13</version>
99
<relativePath />
1010
</parent>
1111

src/main/java/org/ligoj/app/plugin/prov/AbstractProvQuoteInstanceOsResource.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ protected boolean canByol(final VmOs os) {
6363

6464
@Override
6565
protected void saveOrUpdateSpec(final C entity, final E vo) {
66-
entity.setOs(ObjectUtils.defaultIfNull(vo.getOs(), entity.getPrice().getOs()));
66+
entity.setOs(ObjectUtils.getIfNull(vo.getOs(), entity.getPrice().getOs()));
6767
entity.setEphemeral(vo.isEphemeral());
6868
entity.setMaxVariableCost(vo.getMaxVariableCost());
6969
entity.setInternet(vo.getInternet());

src/main/java/org/ligoj/app/plugin/prov/AbstractProvQuoteVmResource.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ protected ProvOptimizer getOptimizer(final ProvQuote configuration, final String
295295
private <G extends INamableBean<?>> G getProfileByName(final G quoteProfile, final String name, List<G> allProfiles,
296296
G defaultProfile) {
297297
if (name == null) {
298-
return ObjectUtils.defaultIfNull(quoteProfile, defaultProfile);
298+
return ObjectUtils.getIfNull(quoteProfile, defaultProfile);
299299
}
300300
return allProfiles.stream().filter(u -> u.getName().equals(name)).findFirst()
301301
.orElseThrow(() -> new EntityNotFoundException(name));
@@ -309,7 +309,7 @@ private <G extends INamableBean<?>> G getProfileByName(final G quoteProfile, fin
309309
* @return The resolved processor requirement. May be <code>null</code>.
310310
*/
311311
protected String getProcessor(final ProvQuote configuration, final String processor) {
312-
return ObjectUtils.defaultIfNull(processor, ObjectUtils.defaultIfNull(configuration.getProcessor(), ""));
312+
return ObjectUtils.getIfNull(processor, ObjectUtils.getIfNull(configuration.getProcessor(), ""));
313313
}
314314

315315
/**
@@ -343,7 +343,7 @@ protected double getCpu(final ProvQuote configuration, final QuoteVm qi) {
343343
* @return The adjusted required RAM from the original one and the RAM configuration.
344344
*/
345345
protected double getRam(final ProvQuote configuration, final QuoteVm qi) {
346-
return Math.max(128, ObjectUtils.defaultIfNull(configuration.getRamAdjustedRate(), 100)
346+
return Math.max(128, ObjectUtils.getIfNull(configuration.getRamAdjustedRate(), 100)
347347
* getReserved(configuration, qi.getRam(), qi.getRamMax()) / 100d);
348348
}
349349

src/main/java/org/ligoj/app/plugin/prov/ProvResource.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,7 @@ public QuoteVo getConfiguration(final Subscription subscription) {
333333
vo.setOptimizer(quote.getOptimizer());
334334
vo.setLicense(quote.getLicense());
335335
vo.setUiSettings(quote.getUiSettings());
336-
vo.setRamAdjustedRate(ObjectUtils.defaultIfNull(quote.getRamAdjustedRate(), 100));
336+
vo.setRamAdjustedRate(ObjectUtils.getIfNull(quote.getRamAdjustedRate(), 100));
337337
vo.setReservationMode(quote.getReservationMode());
338338
vo.setProcessor(quote.getProcessor());
339339
vo.setPhysical(quote.getPhysical());
@@ -415,8 +415,8 @@ public Floating update(@PathParam("subscription") final int subscription, final
415415
var oldUsage = entity.getUsage();
416416
var oldBudget = entity.getBudget();
417417
var oldOptimizer = entity.getOptimizer();
418-
var oldRamAdjusted = ObjectUtils.defaultIfNull(entity.getRamAdjustedRate(), 100);
419-
var oldReservationMode = ObjectUtils.defaultIfNull(entity.getReservationMode(), ReservationMode.RESERVED);
418+
var oldRamAdjusted = ObjectUtils.getIfNull(entity.getRamAdjustedRate(), 100);
419+
var oldReservationMode = ObjectUtils.getIfNull(entity.getReservationMode(), ReservationMode.RESERVED);
420420
var oldProcessor = StringUtils.trimToNull(entity.getProcessor());
421421
var oldPhysical = entity.getPhysical();
422422
entity.setLocation(findLocation(entity.getSubscription().getNode().getId(), vo.getLocation()));
@@ -427,7 +427,7 @@ public Floating update(@PathParam("subscription") final int subscription, final
427427
entity.setOptimizer(Optional.ofNullable(vo.getOptimizer())
428428
.map(u -> findConfiguredByName(optimizerRepository, u, subscription)).orElse(null));
429429
entity.setLicense(vo.getLicense());
430-
entity.setRamAdjustedRate(ObjectUtils.defaultIfNull(vo.getRamAdjustedRate(), 100));
430+
entity.setRamAdjustedRate(ObjectUtils.getIfNull(vo.getRamAdjustedRate(), 100));
431431
entity.setReservationMode(vo.getReservationMode());
432432
entity.setProcessor(StringUtils.trimToNull(vo.getProcessor()));
433433
entity.setPhysical(vo.getPhysical());

src/main/java/org/ligoj/app/plugin/prov/QuoteRelated.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -87,13 +87,13 @@ default <T extends Costed> UpdatedCost newUpdateCost(final RestRepository<T, Int
8787
*/
8888
default <T extends Costed> Floating addCost(final T entity, final Function<T, Floating> costUpdater) {
8989
// Save the previous costs
90-
final double oldCost = ObjectUtils.defaultIfNull(entity.getCost(), 0d);
91-
final double oldMaxCost = ObjectUtils.defaultIfNull(entity.getMaxCost(), 0d);
92-
final double oldCo2 = ObjectUtils.defaultIfNull(entity.getCo2(), 0d);
93-
final double oldMaxCo2 = ObjectUtils.defaultIfNull(entity.getMaxCo2(), 0d);
90+
final double oldCost = ObjectUtils.getIfNull(entity.getCost(), 0d);
91+
final double oldMaxCost = ObjectUtils.getIfNull(entity.getMaxCost(), 0d);
92+
final double oldCo2 = ObjectUtils.getIfNull(entity.getCo2(), 0d);
93+
final double oldMaxCo2 = ObjectUtils.getIfNull(entity.getMaxCo2(), 0d);
9494

95-
final double oldInitial = ObjectUtils.defaultIfNull(entity.getInitialCost(), 0d);
96-
final double oldMaxInitial = ObjectUtils.defaultIfNull(entity.getMaxInitialCost(), 0d);
95+
final double oldInitial = ObjectUtils.getIfNull(entity.getInitialCost(), 0d);
96+
final double oldMaxInitial = ObjectUtils.getIfNull(entity.getMaxInitialCost(), 0d);
9797

9898
// Process the update of this entity
9999
final var newCost = costUpdater.apply(entity);

src/main/java/org/ligoj/app/plugin/prov/catalog/AbstractImportCatalogResource.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
import org.apache.commons.lang3.EnumUtils;
1616
import org.apache.commons.lang3.ObjectUtils;
1717
import org.apache.commons.lang3.StringUtils;
18+
import org.apache.commons.lang3.Strings;
1819
import org.ligoj.app.dao.NodeRepository;
1920
import org.ligoj.app.model.Node;
2021
import org.ligoj.app.plugin.prov.Floating;
@@ -229,7 +230,7 @@ protected <T> Map<String, T> toMap(final String path, final TypeReference<Map<St
229230
}
230231

231232
protected Double toPercent(String raw) {
232-
if (StringUtils.endsWith(raw, "%")) {
233+
if (Strings.CS.endsWith(raw, "%")) {
233234
return Double.valueOf(raw.substring(0, raw.length() - 1));
234235
}
235236

@@ -365,7 +366,7 @@ protected ProvLocation installRegion(final AbstractUpdateContext context, final
365366
r.setLatitude(regionStats.getLatitude());
366367
r.setLongitude(regionStats.getLongitude());
367368
r.setPreferred(regionStats.isPreferred());
368-
r.setDescription(ObjectUtils.defaultIfNull(description, regionStats.getName()));
369+
r.setDescription(ObjectUtils.getIfNull(description, regionStats.getName()));
369370
});
370371
}
371372

@@ -618,7 +619,7 @@ protected double toConversion(final AbstractUpdateContext context, final String
618619
}
619620

620621
private Pattern toPattern(final String regular) {
621-
return Pattern.compile(StringUtils.replace(regular, ".", "\\.") + ".*");
622+
return Pattern.compile(Strings.CS.replace(regular, ".", "\\.") + ".*");
622623
}
623624

624625
/**

src/main/java/org/ligoj/app/plugin/prov/catalog/AbstractUpdateContext.java

Lines changed: 9 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -3,36 +3,18 @@
33
*/
44
package org.ligoj.app.plugin.prov.catalog;
55

6-
import java.util.Collections;
7-
import java.util.HashMap;
8-
import java.util.HashSet;
9-
import java.util.Map;
10-
import java.util.Set;
11-
import java.util.concurrent.ConcurrentHashMap;
12-
import java.util.regex.Pattern;
13-
14-
import org.apache.commons.lang3.StringUtils;
6+
import lombok.Getter;
7+
import lombok.NoArgsConstructor;
8+
import lombok.Setter;
9+
import org.apache.commons.lang3.Strings;
1510
import org.apache.commons.lang3.time.DateUtils;
1611
import org.ligoj.app.model.Node;
1712
import org.ligoj.app.plugin.prov.ProvResource;
18-
import org.ligoj.app.plugin.prov.model.ProvContainerPrice;
19-
import org.ligoj.app.plugin.prov.model.ProvContainerType;
20-
import org.ligoj.app.plugin.prov.model.ProvDatabasePrice;
21-
import org.ligoj.app.plugin.prov.model.ProvDatabaseType;
22-
import org.ligoj.app.plugin.prov.model.ProvFunctionPrice;
23-
import org.ligoj.app.plugin.prov.model.ProvFunctionType;
24-
import org.ligoj.app.plugin.prov.model.ProvInstancePrice;
25-
import org.ligoj.app.plugin.prov.model.ProvInstancePriceTerm;
26-
import org.ligoj.app.plugin.prov.model.ProvInstanceType;
27-
import org.ligoj.app.plugin.prov.model.ProvLocation;
28-
import org.ligoj.app.plugin.prov.model.ProvStoragePrice;
29-
import org.ligoj.app.plugin.prov.model.ProvStorageType;
30-
import org.ligoj.app.plugin.prov.model.ProvSupportPrice;
31-
import org.ligoj.app.plugin.prov.model.ProvSupportType;
13+
import org.ligoj.app.plugin.prov.model.*;
3214

33-
import lombok.Getter;
34-
import lombok.NoArgsConstructor;
35-
import lombok.Setter;
15+
import java.util.*;
16+
import java.util.concurrent.ConcurrentHashMap;
17+
import java.util.regex.Pattern;
3618

3719
/**
3820
* Base context used to perform catalog update.
@@ -310,6 +292,6 @@ public void cleanup() {
310292
* @return The full URL based on the base URL of this context.
311293
*/
312294
public String getUrl(final String relative) {
313-
return baseUrl + StringUtils.prependIfMissing(relative, "/");
295+
return baseUrl + Strings.CS.prependIfMissing(relative, "/");
314296
}
315297
}

src/main/java/org/ligoj/app/plugin/prov/model/VmOs.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,6 @@ public enum VmOs {
8080
* @return the related pricing OS.
8181
*/
8282
public VmOs toPricingOs() {
83-
return ObjectUtils.defaultIfNull(pricingOs, this);
83+
return ObjectUtils.getIfNull(pricingOs, this);
8484
}
8585
}

src/main/java/org/ligoj/app/plugin/prov/quote/database/ProvQuoteDatabaseResource.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import jakarta.ws.rs.core.UriInfo;
2222

2323
import org.apache.commons.lang3.StringUtils;
24+
import org.apache.commons.lang3.Strings;
2425
import org.ligoj.app.plugin.prov.AbstractProvQuoteVmResource;
2526
import org.ligoj.app.plugin.prov.ProvResource;
2627
import org.ligoj.app.plugin.prov.UpdatedCost;
@@ -105,7 +106,7 @@ protected void saveOrUpdateSpec(final ProvQuoteDatabase entity, final QuoteDatab
105106
* @param vPrice The price attribute value.
106107
*/
107108
protected void checkAttribute(final String name, final String pQuote, final String vPrice) {
108-
if (!StringUtils.equalsIgnoreCase(pQuote, vPrice)) {
109+
if (!Strings.CI.equals(pQuote, vPrice)) {
109110
// Incompatible, hack attempt?
110111
log.warn("Attempt to create a database with an incompatible {} {} with catalog {} {}", name, pQuote, name,
111112
vPrice);

src/main/java/org/ligoj/app/plugin/prov/quote/instance/ProvQuoteInstanceResource.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ public UpdatedCost update(final QuoteInstanceEditionVo vo) {
8989
@Override
9090
protected void saveOrUpdateSpec(final ProvQuoteInstance entity, final QuoteInstanceEditionVo vo) {
9191
entity.setSoftware(StringUtils.trimToNull(vo.getSoftware()));
92-
entity.setTenancy(ObjectUtils.defaultIfNull(vo.getTenancy(), ProvTenancy.SHARED));
92+
entity.setTenancy(ObjectUtils.getIfNull(vo.getTenancy(), ProvTenancy.SHARED));
9393
super.saveOrUpdateSpec(entity, vo);
9494
checkOs(entity);
9595
}
@@ -135,7 +135,7 @@ protected List<Object[]> findLowestPrice(final ProvQuote configuration, final Qu
135135
// Resolve the right license model
136136
final var licenseR = normalize(getLicense(configuration, query.getLicense(), os, this::canByol));
137137
final var softwareR = normalize(query.getSoftware());
138-
final var tenancyR = ObjectUtils.defaultIfNull(query.getTenancy(), ProvTenancy.SHARED);
138+
final var tenancyR = ObjectUtils.getIfNull(query.getTenancy(), ProvTenancy.SHARED);
139139
if (optimizer == Optimizer.CO2) {
140140
return ipRepository.findLowestCo2(types, terms, os, location, rate, duration, licenseR, softwareR,
141141
initialCost, tenancyR, PageRequest.of(0, 1));
@@ -155,7 +155,7 @@ protected List<Object[]> findLowestDynamicPrice(final ProvQuote configuration, f
155155
// Resolve the right license model
156156
final var licenseR = normalize(getLicense(configuration, query.getLicense(), os, this::canByol));
157157
final var softwareR = normalize(query.getSoftware());
158-
final var tenancyR = ObjectUtils.defaultIfNull(query.getTenancy(), ProvTenancy.SHARED);
158+
final var tenancyR = ObjectUtils.getIfNull(query.getTenancy(), ProvTenancy.SHARED);
159159
if (optimizer == Optimizer.CO2) {
160160
return ipRepository.findLowestDynamicCo2(types, terms, Math.ceil(Math.max(1, cpu)), gpu,
161161
Math.ceil(round(ram / 1024)), os, location, rate, round(rate * duration), duration, licenseR,

0 commit comments

Comments
 (0)