Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
#JENKINS-17352 Initial commit on adding functionality to manage Weath…
…er & Status columns. Tests were committed, but are going to be uncommitted
  • Loading branch information
ctapobep committed Mar 28, 2013
1 parent 151d129 commit 052d6fe
Show file tree
Hide file tree
Showing 5 changed files with 147 additions and 47 deletions.
67 changes: 67 additions & 0 deletions src/main/java/hudson/plugins/nested_view/AvailableColumns.java
@@ -0,0 +1,67 @@
package hudson.plugins.nested_view;

import com.google.common.collect.Lists;
import hudson.DescriptorExtensionList;
import hudson.model.Descriptor;
import hudson.util.DescribableList;
import hudson.views.ListViewColumn;
import hudson.views.ListViewColumnDescriptor;
import hudson.views.StatusColumn;
import hudson.views.WeatherColumn;
import jenkins.model.Jenkins;
import net.sf.json.JSONObject;
import org.kohsuke.stapler.StaplerRequest;

import java.io.IOException;
import java.util.List;

/**
* @author sbashkyrtsev
*/
public class AvailableColumns {
private DescribableList<ListViewColumn, Descriptor<ListViewColumn>> columns;

public void setColumns(DescribableList<ListViewColumn, Descriptor<ListViewColumn>> columns) {
this.columns = columns;
}

public DescribableList<ListViewColumn, Descriptor<ListViewColumn>> getColumns() {
return columns;
}

public void updateFromForm(StaplerRequest req, JSONObject formData, String key) throws IOException, Descriptor.FormException {
columns.rebuildHetero(req, formData, nestedViewColumns(), key);
}

public boolean isShowStatusColumn() {
return containsColumnWithDescriptor(StatusColumn.DescriptorImpl.class);
}

public boolean isShowWeatherColumn() {
return containsColumnWithDescriptor(WeatherColumn.DescriptorImpl.class);
}

private boolean containsColumnWithDescriptor(Class<? extends ListViewColumnDescriptor> descriptorClass) {
for (ListViewColumn column : columns) {
if (column.getDescriptor().getClass() == descriptorClass) {
return true;
}
}
return false;
}

public static List<Descriptor<ListViewColumn>> nestedViewColumns() {
return extractOptionalColumns(allViewColumns());
}

private static DescriptorExtensionList<ListViewColumn, Descriptor<ListViewColumn>> allViewColumns() {
return Jenkins.getInstance().getDescriptorList(ListViewColumn.class);
}

private static List<Descriptor<ListViewColumn>> extractOptionalColumns(
DescriptorExtensionList<ListViewColumn, Descriptor<ListViewColumn>> extensionList) {
return Lists.<Descriptor<ListViewColumn>>newArrayList(
extensionList.get(WeatherColumn.DescriptorImpl.class),
extensionList.get(StatusColumn.DescriptorImpl.class));
}
}
82 changes: 46 additions & 36 deletions src/main/java/hudson/plugins/nested_view/NestedView.java
Expand Up @@ -24,29 +24,23 @@
*/
package hudson.plugins.nested_view;

import static hudson.Util.fixEmpty;

import hudson.model.*;
import hudson.model.Descriptor.FormException;
import hudson.Extension;
import hudson.Util;
import hudson.model.*;
import hudson.model.Descriptor.FormException;
import hudson.util.DescribableList;
import hudson.util.FormValidation;
import hudson.views.ListViewColumn;
import hudson.views.ViewsTabBar;
import org.kohsuke.stapler.*;
import org.kohsuke.stapler.export.Exported;

import javax.servlet.ServletException;
import java.io.IOException;
import java.util.ArrayDeque;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
import java.util.Deque;
import java.util.LinkedHashSet;
import java.util.List;
import java.util.Set;
import java.util.*;
import java.util.concurrent.CopyOnWriteArrayList;

import org.kohsuke.stapler.*;
import org.kohsuke.stapler.export.Exported;
import static hudson.Util.fixEmpty;

/**
* View type that contains only another set of views.
Expand All @@ -68,6 +62,7 @@ public class NestedView extends View implements ViewGroup, StaplerProxy {
* Name of the subview to show when this tree view is selected. May be null/empty.
*/
private String defaultView;
private AvailableColumns columns;

@DataBoundConstructor
public NestedView(String name) {
Expand Down Expand Up @@ -101,11 +96,11 @@ public Item doCreateItem(StaplerRequest req, StaplerResponse rsp)
throws IOException, ServletException {
ItemGroup itemGroup = getItemGroup();
if (itemGroup instanceof ModifiableItemGroup) {
return ((ModifiableItemGroup)itemGroup).doCreateItem(req, rsp);
return ((ModifiableItemGroup) itemGroup).doCreateItem(req, rsp);
}
return null;
}

/**
* Checks if a nested view with the given name exists.
*/
Expand All @@ -114,18 +109,25 @@ public FormValidation doViewExistsCheck(@QueryParameter String value) {

String view = fixEmpty(value);
return (view == null || getView(view) == null) ? FormValidation.ok()
: FormValidation.error(hudson.model.Messages.Hudson_ViewAlreadyExists(view));
: FormValidation.error(hudson.model.Messages.Hudson_ViewAlreadyExists(view));
}

@Override
public synchronized void onJobRenamed(Item item, String oldName, String newName) {
// forward to children
for (View v : views)
v.onJobRenamed(item,oldName,newName);
v.onJobRenamed(item, oldName, newName);
}

protected void submit(StaplerRequest req) throws IOException, ServletException, FormException {
protected synchronized void submit(StaplerRequest req) throws IOException, ServletException, FormException {
defaultView = Util.fixEmpty(req.getParameter("defaultView"));
if (columns == null) {
columns = new AvailableColumns();
}
if (columns.getColumns() == null) {
columns.setColumns(new DescribableList<ListViewColumn, Descriptor<ListViewColumn>>(this));
}
columns.updateFromForm(req, req.getSubmittedForm(), "columnsToShow");
}

public boolean canDelete(View view) {
Expand Down Expand Up @@ -156,6 +158,10 @@ public View getDefaultView() {
return isDefault() ? null : getView(defaultView);
}

public AvailableColumns getColumnsToShow() {
return columns;
}

public void onViewRenamed(View view, String oldName, String newName) {
// noop
}
Expand Down Expand Up @@ -183,7 +189,7 @@ void setOwner(ViewGroup owner) {

/**
* Returns the worst result for this nested view.
*
* <p/>
* <p>To get the worst result, this method browses all the jobs this view
* contains. Also, as soon as it finds the worst result possible (cf.
* {@link #WORST_RESULT}), the browsing stops.</p>
Expand All @@ -200,10 +206,9 @@ public Result getWorstResult() {
List<NestedView> nestedViews = new ArrayList<NestedView>();

for (View v : views) {
if(v instanceof NestedView) {
if (v instanceof NestedView) {
nestedViews.add((NestedView) v);
}
else {
} else {
normalViews.add(v);
}
}
Expand Down Expand Up @@ -257,8 +262,8 @@ private static Result getWorstResultForNormalView(View v) {
Result result = Result.SUCCESS, check;
for (TopLevelItem item : v.getItems()) {
if (item instanceof Job && !( // Skip disabled projects
item instanceof AbstractProject && ((AbstractProject)item).isDisabled())) {
final Run lastCompletedBuild = ((Job)item).getLastCompletedBuild();
item instanceof AbstractProject && ((AbstractProject) item).isDisabled())) {
final Run lastCompletedBuild = ((Job) item).getLastCompletedBuild();
if (lastCompletedBuild != null) {
found = true;
if ((check = lastCompletedBuild.getResult()).isWorseThan(result)) {
Expand All @@ -277,6 +282,7 @@ private static Result getWorstResultForNormalView(View v) {
/**
* Returns the worst result for a view, wether is a normal view or a nested
* one.
*
* @see #getWorstResult()
* @see #getWorstResultForNormalView(hudson.model.View)
*/
Expand All @@ -290,7 +296,7 @@ public static Result getWorstResult(View v) {

/**
* Returns the health of this nested view.
*
* <p/>
* <p>Notice that, if a job is contained in several sub-views of the current
* view, then it is taken into account only once to get accurate stats.</p>
* <p>This algorithm has been derecursified, hence the stack stuff.</p>
Expand All @@ -309,23 +315,22 @@ public HealthReportContainer getHealth() {
for (View v : ((NestedView) currentView).views) {
viewsStack.push(v);
}
}
else {
} else {
items.addAll(currentView.getItems());
}
} while (!viewsStack.isEmpty());

HealthReportContainer hrc = new HealthReportContainer();
for (TopLevelItem item : items) {
if (item instanceof Job) {
hrc.sum += ((Job)item).getBuildHealth().getScore();
hrc.sum += ((Job) item).getBuildHealth().getScore();
hrc.count++;
}
}

hrc.report = hrc.count > 0
? new HealthReport(hrc.sum / hrc.count, Messages._ViewHealth(hrc.count))
: new HealthReport(100, Messages._NoJobs());
? new HealthReport(hrc.sum / hrc.count, Messages._ViewHealth(hrc.count))
: new HealthReport(100, Messages._NoJobs());

return hrc;
}
Expand All @@ -344,21 +349,21 @@ private static HealthReportContainer getHealthForNormalView(View view) {
}
}
hrc.report = hrc.count > 0
? new HealthReport(hrc.sum / hrc.count, Messages._ViewHealth(hrc.count))
: null;
? new HealthReport(hrc.sum / hrc.count, Messages._ViewHealth(hrc.count))
: null;
return hrc;
}

/**
* Returns the health of a view, wether it is a normal or a nested one.
*
* @see #getHealth()
* @see #getHealthForNormalView(hudson.model.View)
*/
public static HealthReportContainer getViewHealth(View v) {
if (v instanceof NestedView) {
return ((NestedView) v).getHealth();
}
else {
} else {
return getHealthForNormalView(v);
}
}
Expand All @@ -374,10 +379,14 @@ public ViewsTabBar getViewsTabBar() {
public static class HealthReportContainer {
private HealthReport report;
private int sum = 0, count = 0;
private HealthReportContainer() { }

private HealthReportContainer() {
}

public HealthReport getBuildHealth() {
return report;
}

public List<HealthReport> getBuildHealthReports() {
return report != null ? Collections.singletonList(report) : Collections.<HealthReport>emptyList();
}
Expand All @@ -404,5 +413,6 @@ public static final class DescriptorImpl extends ViewDescriptor {
public String getDisplayName() {
return Messages.DisplayName();
}

}
}
Expand Up @@ -23,6 +23,7 @@ THE SOFTWARE.
-->
<?jelly escape-by-default='true'?>
<j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form">
<!--<f:checkbox default="false" checked="${it.showWeatherColumn}" title="Show Weather Column" name="showWeatherColumn"/>-->
<j:if test="${!it.isDefault()}">
<f:entry title="${%Default subview}" help="/plugin/nested-view/help-defaultView.html">
<select class="setting-input" name="defaultView">
Expand All @@ -38,4 +39,13 @@ THE SOFTWARE.
</select>
</f:entry>
</j:if>
<f:section title="${%Columns}">
<j:invokeStatic var="allColumns" className="hudson.plugins.nested_view.AvailableColumns" method="nestedViewColumns"/>
<f:block>
<f:hetero-list name="columnsToShow" hasHeader="true"
descriptors="${allColumns}"
items="${it.columnsToShow.columns}"
addCaption="${%Add column}"/>
</f:block>
</f:section>
</j:jelly>
32 changes: 21 additions & 11 deletions src/main/resources/hudson/plugins/nested_view/NestedView/main.jelly
Expand Up @@ -22,20 +22,22 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
THE SOFTWARE.
-->
<?jelly escape-by-default='true'?>
<j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form">
<j:jelly xmlns:j="jelly:core" xmlns:st="jelly:stapler"
xmlns:t="/lib/hudson">
<t:setIconSize/>
<div class="dashboard">
<j:set var="views" value="${it.owner.views}" />
<j:set var="currentView" value="${it}" />
<st:include page="viewTabs.jelly" it="${it.viewsTabBar}" />
<j:set var="views" value="${it.owner.views}"/>
<j:set var="currentView" value="${it}"/>
<j:set var="optionalColumns" value="${it.columnsToShow}"/>
<st:include page="viewTabs.jelly" it="${it.viewsTabBar}"/>
<j:getStatic var="grey" className="hudson.model.BallColor" field="GREY"/>

<j:choose>
<j:when test="${empty(it.views)}">
<p>
${%This view has no subviews associated with it.}
<j:if test="${it.hasPermission(it.CONFIGURE)}">
<j:out value="${%AddSome('newView')}" />
<j:out value="${%AddSome('newView')}"/>
</j:if>
</p>
</j:when>
Expand All @@ -46,19 +48,27 @@ THE SOFTWARE.
style="margin-top:0; border-top:none">
<tr style="border-top:0">
<th width="1"></th>
<th width="1" tooltip="${%StatusTooltip}">${h.nbspIndent(iconSize)}S</th>
<th width="1" tooltip="${%WeatherTooltip}">${h.nbspIndent(iconSize)}W</th>
<j:if test="${optionalColumns.showStatusColumn}">
<th width="1" tooltip="${%StatusTooltip}">${h.nbspIndent(iconSize)}S</th>
</j:if>
<j:if test="${optionalColumns.showWeatherColumn}">
<th width="1" tooltip="${%WeatherTooltip}">${h.nbspIndent(iconSize)}W</th>
</j:if>
<th width="*">${%View}</th>
</tr>
<j:forEach var="v" items="${it.views}">
<tr>
<td style="padding-left:5px;width:20px">
<img src="${imagesURL}/${iconSize}/folder.gif" alt="" />
<img src="${imagesURL}/${iconSize}/folder.gif" alt=""/>
</td>
<j:set var="result" value="${it.getWorstResult(v)}"/>
<t:ballColorTd it="${result!=null ? result.color : grey}"/>
<j:if test="${optionalColumns.showStatusColumn}">
<j:set var="result" value="${it.getWorstResult(v)}"/>
<t:ballColorTd it="${result!=null ? result.color : grey}"/>
</j:if>
<!-- Use a fake "job" to provide BuildHealth -->
<t:buildHealth job="${it.getViewHealth(v)}" td="true"/>
<j:if test="${optionalColumns.showWeatherColumn}">
<t:buildHealth job="${it.getViewHealth(v)}" td="true"/>
</j:if>
<td>
<a href="${rootURL}/${v.url}">${v.viewName}</a>
</td>
Expand Down
3 changes: 3 additions & 0 deletions src/test/java/hudson/plugins/nested_view/NestedViewTest.java
Expand Up @@ -34,13 +34,16 @@
import hudson.util.FormValidation;
import static hudson.util.FormValidation.Kind.*;
import java.util.List;

import org.junit.Ignore;
import org.jvnet.hudson.test.FailureBuilder;
import org.jvnet.hudson.test.HudsonTestCase;

/**
* Test interaction of nested-view plugin with Jenkins core.
* @author Alan Harder
*/
@Ignore
public class NestedViewTest extends HudsonTestCase {

public void test() throws Exception {
Expand Down

0 comments on commit 052d6fe

Please sign in to comment.