Permalink
Browse files

[FIXED JENKINS-16075] Remove redundant resource path from icon URL.

Apparently the fix for JENKINS-16530 now includes this automatically
for all plugin images, regardless of whether the plugin was already
doing the right thing and allowing caching.
  • Loading branch information...
orrc committed Apr 10, 2013
1 parent 16db36f commit 7d90c1de9206daa9f3d13961687e4cf7e40b0ac9
Showing with 1 addition and 2 deletions.
  1. +1 −2 src/main/java/org/jenkinsci/plugins/android_lint/LintDescriptor.java
@@ -1,7 +1,6 @@
package org.jenkinsci.plugins.android_lint;

import hudson.Extension;
import hudson.Functions;
import hudson.plugins.analysis.core.PluginDescriptor;

/**
@@ -17,7 +16,7 @@
public static final String PLUGIN_NAME = "androidLint";

/** Used to specify location of resources. */
public static final String PLUGIN_ROOT = Functions.getResourcePath() + "/plugin/android-lint/";
public static final String PLUGIN_ROOT = "/plugin/android-lint/";

/** Icon to use for the result and project action. */
private static final String ACTION_ICON = PLUGIN_ROOT + "icons/android-24x24.png";

2 comments on commit 7d90c1d

@jglick

This comment has been minimized.

Copy link
Member

jglick replied Apr 11, 2013

Well, at least for icons used from <l:task> in recent Jenkins releases.

@orrc

This comment has been minimized.

Copy link
Member

orrc replied Apr 11, 2013

True, though I hope that pretty much all plugin images are now cached automatically by core, as I've had to undo the caching I eagerly added in my plugins now a few times now! :)

Please sign in to comment.