Skip to content

Commit 5855f07

Browse files
committed
fixes #377: Remove erroneous index confirmation message
The archiving settings page on the admin console erroneously displays "Search Indexes are rebuilding." all the time - even when message archiving isn't even enabled. I'm actually unsure why this change works. I assume there's javascript/CSS magic that causes the problem.
1 parent 63e84fe commit 5855f07

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

changelog.html

+1
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ <h1>
4646

4747
<p><b>2.5.1</b> -- (tbd)</p>
4848
<ul>
49+
<li>[<a href='https://github.com/igniterealtime/openfire-monitoring-plugin/issues/377'>Issue #377</a>] - Fixes: Erroneous index confirmation message</li>
4950
<li>[<a href='https://github.com/igniterealtime/openfire-monitoring-plugin/issues/367'>Issue #367</a>] - Fixes: JRobin's repository uses invalid certificate</li>
5051
<li>[<a href='https://github.com/igniterealtime/openfire-monitoring-plugin/issues/249'>Issue #249</a>] - Fixes: Erroneous index rebuild progress indicator</li>
5152
</ul>

src/web/archiving-settings.jsp

+4-1
Original file line numberDiff line numberDiff line change
@@ -306,10 +306,13 @@
306306
}
307307
%>
308308

309-
<div class="success" id="rebuild" style="display: <%=rebuildIndex?"block":"none"%>">
309+
310+
<% if (rebuildIndex) { %>
311+
<div class="success" id="rebuild">
310312
<fmt:message key="archive.settings.rebuild.success"/>
311313
<br/>
312314
</div>
315+
<% } %>
313316

314317
<% if (errors.size() > 0) { %>
315318
<div class="error">

0 commit comments

Comments
 (0)