Skip to content

Commit 7222779

Browse files
Use resource Id
1 parent 17c436e commit 7222779

File tree

6 files changed

+17
-29
lines changed

6 files changed

+17
-29
lines changed

engine/schema/src/main/java/com/cloud/dc/ClusterDetailsDaoImpl.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,11 @@ public class ClusterDetailsDaoImpl extends ResourceDetailsDaoBase<ClusterDetails
4343

4444
protected ClusterDetailsDaoImpl() {
4545
ClusterSearch = createSearchBuilder();
46-
ClusterSearch.and("clusterId", ClusterSearch.entity().getClusterId(), SearchCriteria.Op.EQ);
46+
ClusterSearch.and("clusterId", ClusterSearch.entity().getResourceId(), SearchCriteria.Op.EQ);
4747
ClusterSearch.done();
4848

4949
DetailSearch = createSearchBuilder();
50-
DetailSearch.and("clusterId", DetailSearch.entity().getClusterId(), SearchCriteria.Op.EQ);
50+
DetailSearch.and("clusterId", DetailSearch.entity().getResourceId(), SearchCriteria.Op.EQ);
5151
DetailSearch.and("name", DetailSearch.entity().getName(), SearchCriteria.Op.EQ);
5252
DetailSearch.done();
5353
}

engine/schema/src/main/java/com/cloud/dc/ClusterDetailsVO.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public class ClusterDetailsVO implements ResourceDetail {
3535
private long id;
3636

3737
@Column(name = "cluster_id")
38-
private long clusterId;
38+
private long resourceId;
3939

4040
@Column(name = "name")
4141
private String name;
@@ -47,18 +47,14 @@ protected ClusterDetailsVO() {
4747
}
4848

4949
public ClusterDetailsVO(long clusterId, String name, String value) {
50-
this.clusterId = clusterId;
50+
this.resourceId = clusterId;
5151
this.name = name;
5252
this.value = value;
5353
}
5454

55-
public long getClusterId() {
56-
return clusterId;
57-
}
58-
5955
@Override
6056
public long getResourceId() {
61-
return clusterId;
57+
return resourceId;
6258
}
6359

6460
public String getName() {

engine/schema/src/main/java/com/cloud/domain/DomainDetailVO.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public class DomainDetailVO implements ResourceDetail {
3434
private long id;
3535

3636
@Column(name = "domain_id")
37-
private long domainId;
37+
private long resourceId;
3838

3939
@Column(name = "name")
4040
private String name;
@@ -46,18 +46,14 @@ protected DomainDetailVO() {
4646
}
4747

4848
public DomainDetailVO(long domainId, String name, String value) {
49-
this.domainId = domainId;
49+
this.resourceId = domainId;
5050
this.name = name;
5151
this.value = value;
5252
}
5353

54-
public long getDomainId() {
55-
return domainId;
56-
}
57-
5854
@Override
5955
public long getResourceId() {
60-
return domainId;
56+
return resourceId;
6157
}
6258

6359
public String getName() {

engine/schema/src/main/java/com/cloud/domain/dao/DomainDetailsDaoImpl.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,14 +45,14 @@ public class DomainDetailsDaoImpl extends ResourceDetailsDaoBase<DomainDetailVO>
4545

4646
protected DomainDetailsDaoImpl() {
4747
domainSearch = createSearchBuilder();
48-
domainSearch.and("domainId", domainSearch.entity().getDomainId(), Op.EQ);
48+
domainSearch.and("domainId", domainSearch.entity().getResourceId(), Op.EQ);
4949
domainSearch.done();
5050
}
5151

5252
@Override
5353
public Map<String, String> findDetails(long domainId) {
5454
QueryBuilder<DomainDetailVO> sc = QueryBuilder.create(DomainDetailVO.class);
55-
sc.and(sc.entity().getDomainId(), Op.EQ, domainId);
55+
sc.and(sc.entity().getResourceId(), Op.EQ, domainId);
5656
List<DomainDetailVO> results = sc.list();
5757
Map<String, String> details = new HashMap<String, String>(results.size());
5858
for (DomainDetailVO r : results) {
@@ -78,7 +78,7 @@ public void persist(long domainId, Map<String, String> details) {
7878
@Override
7979
public DomainDetailVO findDetail(long domainId, String name) {
8080
QueryBuilder<DomainDetailVO> sc = QueryBuilder.create(DomainDetailVO.class);
81-
sc.and(sc.entity().getDomainId(), Op.EQ, domainId);
81+
sc.and(sc.entity().getResourceId(), Op.EQ, domainId);
8282
sc.and(sc.entity().getName(), Op.EQ, name);
8383
return sc.find();
8484
}

engine/schema/src/main/java/com/cloud/user/AccountDetailVO.java

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public class AccountDetailVO implements ResourceDetail {
3434
private long id;
3535

3636
@Column(name = "account_id")
37-
private long accountId;
37+
private long resourceId;
3838

3939
@Column(name = "name")
4040
private String name;
@@ -46,18 +46,14 @@ protected AccountDetailVO() {
4646
}
4747

4848
public AccountDetailVO(long accountId, String name, String value) {
49-
this.accountId = accountId;
49+
this.resourceId = accountId;
5050
this.name = name;
5151
this.value = value;
5252
}
5353

54-
public long getAccountId() {
55-
return accountId;
56-
}
57-
5854
@Override
5955
public long getResourceId() {
60-
return accountId;
56+
return resourceId;
6157
}
6258

6359
public String getName() {

engine/schema/src/main/java/com/cloud/user/AccountDetailsDaoImpl.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,14 +55,14 @@ public class AccountDetailsDaoImpl extends ResourceDetailsDaoBase<AccountDetailV
5555

5656
protected AccountDetailsDaoImpl() {
5757
accountSearch = createSearchBuilder();
58-
accountSearch.and("accountId", accountSearch.entity().getAccountId(), Op.EQ);
58+
accountSearch.and("accountId", accountSearch.entity().getResourceId(), Op.EQ);
5959
accountSearch.done();
6060
}
6161

6262
@Override
6363
public Map<String, String> findDetails(long accountId) {
6464
QueryBuilder<AccountDetailVO> sc = QueryBuilder.create(AccountDetailVO.class);
65-
sc.and(sc.entity().getAccountId(), Op.EQ, accountId);
65+
sc.and(sc.entity().getResourceId(), Op.EQ, accountId);
6666
List<AccountDetailVO> results = sc.list();
6767
Map<String, String> details = new HashMap<String, String>(results.size());
6868
for (AccountDetailVO r : results) {
@@ -88,7 +88,7 @@ public void persist(long accountId, Map<String, String> details) {
8888
@Override
8989
public AccountDetailVO findDetail(long accountId, String name) {
9090
QueryBuilder<AccountDetailVO> sc = QueryBuilder.create(AccountDetailVO.class);
91-
sc.and(sc.entity().getAccountId(), Op.EQ, accountId);
91+
sc.and(sc.entity().getResourceId(), Op.EQ, accountId);
9292
sc.and(sc.entity().getName(), Op.EQ, name);
9393
return sc.find();
9494
}

0 commit comments

Comments
 (0)