Skip to content

Commit fc95fd9

Browse files
committed
Split Property into StandardProperty and Abstract for cleanup
Signed-off-by: Stefan Bischof <stbischof@bipolis.org>
1 parent 8e5a448 commit fc95fd9

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

47 files changed

+973
-1300
lines changed
Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,71 @@
1+
package mondrian.olap;
2+
3+
import org.eclipse.daanse.olap.api.element.Property;
4+
5+
import mondrian.spi.PropertyFormatter;
6+
7+
public abstract class AbstractProperty implements Property {
8+
9+
private String name;
10+
private String description;
11+
12+
private final Datatype type;
13+
private final boolean internal;
14+
private final boolean member;
15+
private final boolean cell;
16+
17+
protected AbstractProperty(String name, Datatype type, boolean internal, boolean member, boolean cell,
18+
String description) {
19+
this.name = name;
20+
this.description = description;
21+
this.type = type;
22+
this.internal = internal;
23+
this.member = member;
24+
this.cell = cell;
25+
}
26+
27+
@Override
28+
public String getCaption() {
29+
return name;
30+
}
31+
32+
@Override
33+
public String getDescription() {
34+
return description;
35+
}
36+
37+
@Override
38+
public PropertyFormatter getFormatter() {
39+
return null;
40+
}
41+
42+
@Override
43+
public String getName() {
44+
return name;
45+
}
46+
47+
@Override
48+
public Datatype getType() {
49+
return type;
50+
}
51+
52+
@Override
53+
public boolean isCellProperty() {
54+
return cell;
55+
}
56+
57+
@Override
58+
public boolean isInternal() {
59+
return internal;
60+
}
61+
62+
@Override
63+
public boolean isMemberProperty() {
64+
return member;
65+
}
66+
67+
@Override
68+
public String toString() {
69+
return name;
70+
}
71+
}

mondrian/src/main/java/mondrian/olap/FormulaImpl.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -160,9 +160,9 @@ public void accept(Validator validator) {
160160
Expression formatExp = getFormatExp(validator);
161161
if (formatExp != null) {
162162
mdxMember.setProperty(
163-
Property.FORMAT_EXP_PARSED.name, formatExp);
163+
StandardProperty.FORMAT_EXP_PARSED.getName(), formatExp);
164164
mdxMember.setProperty(
165-
Property.FORMAT_EXP.name, Util.unparse(formatExp));
165+
StandardProperty.FORMAT_EXP.getName(), Util.unparse(formatExp));
166166
}
167167

168168
final List<MemberProperty> memberPropertyList =
@@ -173,7 +173,7 @@ public void accept(Validator validator) {
173173
for (int i = 0; i < memberPropertyList.size(); i++) {
174174
MemberProperty memberProperty = memberPropertyList.get(i);
175175
if (memberProperty.getName().equals(
176-
Property.CELL_FORMATTER_SCRIPT_LANGUAGE.name))
176+
StandardProperty.CELL_FORMATTER_SCRIPT_LANGUAGE.getName()))
177177
{
178178
memberPropertyList.remove(i);
179179
memberPropertyList.add(0, memberProperty);
@@ -183,7 +183,7 @@ public void accept(Validator validator) {
183183
// For each property of the formula, make it a property of the
184184
// member.
185185
for (MemberProperty memberProperty : memberPropertyList) {
186-
if (Property.FORMAT_PROPERTIES.contains(
186+
if (StandardProperty.FORMAT_PROPERTIES.contains(
187187
memberProperty.getName()))
188188
{
189189
continue; // we already dealt with format_string props
@@ -454,7 +454,7 @@ public Member getMdxMember() {
454454
*/
455455
@Override
456456
public Number getSolveOrder() {
457-
return getIntegerMemberProperty(Property.SOLVE_ORDER.name);
457+
return getIntegerMemberProperty(StandardProperty.SOLVE_ORDER.getName());
458458
}
459459

460460
/**
@@ -515,7 +515,7 @@ private static Number quickEval(Expression exp) {
515515
private Expression getFormatExp(Validator validator) {
516516
// If they have specified a format string (which they can do under
517517
// several names) return that.
518-
for (String prop : Property.FORMAT_PROPERTIES) {
518+
for (String prop : StandardProperty.FORMAT_PROPERTIES) {
519519
Expression formatExp = getMemberProperty(prop);
520520
if (formatExp != null) {
521521
return formatExp;
@@ -686,7 +686,7 @@ private void returnFormula(Member member) {
686686

687687
private Expression getFormula(Member member) {
688688
return (Expression)
689-
member.getPropertyValue(Property.FORMAT_EXP_PARSED.name);
689+
member.getPropertyValue(StandardProperty.FORMAT_EXP_PARSED.getName());
690690
}
691691
}
692692
}

mondrian/src/main/java/mondrian/olap/MemberBase.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ public MemberType getMemberType() {
156156

157157
@Override
158158
public String getDescription() {
159-
return (String) getPropertyValue( Property.DESCRIPTION_PROPERTY.name );
159+
return (String) getPropertyValue( StandardProperty.DESCRIPTION_PROPERTY.getName() );
160160
}
161161

162162
@Override

0 commit comments

Comments
 (0)