Skip to content

Commit 102f3d0

Browse files
authored
Merge pull request #423 from wttech/reformat-summary
reformat summary entries
2 parents 511d1c9 + e447e9b commit 102f3d0

File tree

5 files changed

+8
-23
lines changed

5 files changed

+8
-23
lines changed

app/aem/api/src/main/java/com/cognifide/apm/api/services/ExecutionResult.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,8 @@ interface Entry {
4242

4343
List<String> getMessages();
4444

45-
List<String> getPrettyMessages();
46-
4745
List<String> getParameters();
4846

49-
List<String> getPrettyParameters();
50-
5147
Status getStatus();
5248
}
5349
}

app/aem/core/src/main/java/com/cognifide/apm/core/logger/ProgressEntry.java

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@
2626
import com.google.common.collect.ImmutableList;
2727
import java.util.Collections;
2828
import java.util.List;
29-
import java.util.stream.Collectors;
3029

3130
public class ProgressEntry implements Entry {
3231

@@ -71,25 +70,11 @@ public List<String> getMessages() {
7170
return messages;
7271
}
7372

74-
@Override
75-
public List<String> getPrettyMessages() {
76-
return messages.stream()
77-
.map(message -> message.replaceAll("\n", "<br>").replaceAll("\t", "&nbsp;&nbsp;&nbsp;&nbsp;"))
78-
.collect(Collectors.toList());
79-
}
80-
8173
@Override
8274
public List<String> getParameters() {
8375
return parameters;
8476
}
8577

86-
@Override
87-
public List<String> getPrettyParameters() {
88-
return parameters.stream()
89-
.map(message -> message.replaceAll("\n", "<br>").replaceAll("\t", "&nbsp;&nbsp;&nbsp;&nbsp;"))
90-
.collect(Collectors.toList());
91-
}
92-
9378
@Override
9479
public Status getStatus() {
9580
return status;

app/aem/ui.apps.base/src/main/content/jcr_root/apps/apm/clientlibs/views/summary/css/styles.less

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,4 +23,8 @@
2323
background-color: rgba(221, 52, 52, 0.7);
2424
}
2525
}
26-
}
26+
}
27+
28+
li {
29+
white-space: pre-wrap;
30+
}

app/aem/ui.apps.base/src/main/content/jcr_root/apps/apm/components/reference/reference.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ <h3><a name="available-commands">Available commands</a></h3>
5050
<p>${map.description}</p>
5151
<div data-sly-test="${map.examples}" class="code-example">
5252
<strong>Examples</strong>
53-
<code data-sly-repeat.example="${map.examples}">${example @ context='html'}</code>
53+
<code data-sly-repeat.example="${map.examples}">${example}</code>
5454
</div>
5555
</td>
5656
</tr>

app/aem/ui.apps.base/src/main/content/jcr_root/apps/apm/components/summaryTable/summaryTable.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,15 +55,15 @@
5555
<td class="_coral-Table-cell">${execution.command.toUpperCase}</td>
5656
<td class="_coral-Table-cell">
5757
<ul>
58-
<li data-sly-repeat.text="${execution.prettyParameters}">${text @ context='html'}</li>
58+
<li data-sly-repeat.text="${execution.parameters}">${text}</li>
5959
</ul>
6060
</td>
6161
<td class="_coral-Table-cell">
6262
<coral-icon title="${execution.status.iconTitle}" icon="${execution.status.iconType}"/>
6363
</td>
6464
<td class="_coral-Table-cell">
6565
<ul>
66-
<li data-sly-repeat.text="${execution.prettyMessages}">${text @ context='html'}</li>
66+
<li data-sly-repeat.text="${execution.messages}">${text}</li>
6767
</ul>
6868
</td>
6969
</tr>

0 commit comments

Comments
 (0)