Fair point, though not the issue I was fixing. If you want to update the
pull request or create a second...

On Sunday, 25 May 2014, Oliver Gondža <notifications@github.com> wrote:

> In
> src/main/java/org/jenkinsci/plugins/mailwatcher/WatcherItemListener.java:
>
> > @@ -153,7 +155,19 @@ public void send(final Object o) {
> >                      recipients(property.getWatcherAddresses());
> >                  }
> >
> > -                url(job.getShortUrl());
> > +                Stack<String> stack = new Stack<String>();
> > +                stack.push(job.getShortUrl());
> > +                ItemGroup parent = job.getParent();
> > +                while (parent != null && parent instanceof Item) {
> > +                    Item item = (Item) parent;
> > +                    stack.push(item.getShortUrl());
> > +                    parent = item.getParent();
> > +                }
> > +                StringBuilder urlPath = new StringBuilder();
> > +                while (!stack.isEmpty()) {
> > +                    urlPath.append(stack.pop());
> > +                }
> > +                url(urlPath.toString());
> >                  name(job.getName());
>
> I mean for the name, not url. s/name(job.getName())/
> name(job.getFullName())/.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/jenkinsci/mail-watcher-plugin/pull/1/files#r13032779>
> .
>


-- 
Sent from my phone