The basic fix is to use ${rootURL} plus full model object URLs rather than relying on ${jobBaseUrl}.
This is made trickier by the fact that the model object URLs are computed inside ProgressiveRendering.compute,
and therefore will not be correct when nondefault views are in the crumb trail unless the original request information is present.
So modifying ProgressiveRendering to preserve a copy of the original request for use during computation.
(This could probably be used to simplify parts of AsynchPeople as well.)
Also improving AbstractItem.getUrl to properly construct a URL including views even when the current page is not inside the item;
it should suffice for some ancestor of the current item (or a view thereof) to be in the ancestor list of this page.

Corrects 9ce6b0fe.

Existing code referring to a member of type AbstractBuild often called methods not present in Run.
Changed to retain the existing fields and getters but deprecating them and introducing replacements.

Fix of JENKINS-8408 broke some tests of workspace-based polling; disable the fix when inside a test, for better predictability.
(Ideally Jenkins would actually detect whether there was a *plan* to connect a slave of a given name.)

Documented this feature and integrated a newer version of Stapler.

I took the opportunity to reorder the content in the section, so that
the tree parameter becomes the main subject.

At this point, the depth parameter should be only used in a limited
circumstance, so it should be treated as such.

I came into this use case while working on the workflow, but I think it's useful in other context as well,
such as Scriptler plugin needing to define some parameters.

Make sure the RepositoryBrowser is associated with a ChangeLogSet so we do not need to go back through the build and SCM to find it.

Added annotations to kernel security classes.

Most of methods define @Nonnull inputs and outputs (the legacy behavior is mostly non-documented), but there're several exceptions

Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>

[FIX JENKINS-16827] Don't try to guess the build number

[READY] Additional null checks for node access methods.

Fixed node handling methods in order to comply with https://github.com/jenkinsci/jenkins/commit/3f8fc4e2ec5b1b421e1551a2a2fc2a284be330d1



Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>

Cf.: https://github.com/jenkinsci/remote-terminal-access-plugin/commit/db24e2126b649689652ac8ddb100f3ef6680a422

Annotate hudson.Util methods by @CheckForNull and @Nonnull

Add logging to PeepholePermalink's RunListener

Annotated hudson.model.Build and hudson.model.Run, minor fixes in Javadoc

Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>

[FIXED JENKINS-20559 and JENKINS-19454] - hudson.Launcher bugfix and additional wrapper

Signed-off-by: Oleg Nenashev <o.v.nenashev@gmail.com>