@@ -113,11 +113,11 @@ public abstract class BasePage extends SessionPage { | |||
return canonicalUrl; | |||
} | |||
protected void manualRedirect(Class<? extends BasePage> pageClass) { | |||
redirect(pageClass, null); | |||
protected void redirectTo(Class<? extends BasePage> pageClass) { | |||
redirectTo(pageClass, null); | |||
} | |||
protected void redirect(Class<? extends BasePage> pageClass, PageParameters parameters) { | |||
protected void redirectTo(Class<? extends BasePage> pageClass, PageParameters parameters) { | |||
String absoluteUrl = getCanonicalUrl(pageClass, parameters); | |||
getRequestCycle().setRequestTarget(new RedirectRequestTarget(absoluteUrl)); | |||
} |
@@ -143,7 +143,7 @@ public class EditMilestonePage extends RepositoryPage { | |||
} | |||
if (success && app().tickets().updateMilestone(getRepositoryModel(), tm, createdBy)) { | |||
redirect(TicketsPage.class, WicketUtils.newOpenTicketsParameter(repositoryName)); | |||
redirectTo(TicketsPage.class, WicketUtils.newOpenTicketsParameter(repositoryName)); | |||
} else { | |||
// TODO error | |||
} |
@@ -331,13 +331,13 @@ public class EditTicketPage extends RepositoryPage { | |||
if (ticket != null) { | |||
TicketNotifier notifier = app().tickets().createNotifier(); | |||
notifier.sendMailing(ticket); | |||
redirect(TicketsPage.class, WicketUtils.newObjectParameter(getRepositoryModel().name, "" + ticket.number)); | |||
redirectTo(TicketsPage.class, WicketUtils.newObjectParameter(getRepositoryModel().name, "" + ticket.number)); | |||
} else { | |||
// TODO error | |||
} | |||
} else { | |||
// nothing to change?! | |||
redirect(TicketsPage.class, WicketUtils.newObjectParameter(getRepositoryModel().name, "" + ticket.number)); | |||
redirectTo(TicketsPage.class, WicketUtils.newObjectParameter(getRepositoryModel().name, "" + ticket.number)); | |||
} | |||
} | |||
}); |
@@ -108,7 +108,7 @@ public class NewMilestonePage extends RepositoryPage { | |||
if (milestone != null) { | |||
milestone.due = due; | |||
app().tickets().updateMilestone(getRepositoryModel(), milestone, createdBy); | |||
redirect(TicketsPage.class, WicketUtils.newOpenTicketsParameter(repositoryName)); | |||
redirectTo(TicketsPage.class, WicketUtils.newOpenTicketsParameter(repositoryName)); | |||
} else { | |||
// TODO error | |||
} |
@@ -223,7 +223,7 @@ public class NewTicketPage extends RepositoryPage { | |||
TicketNotifier notifier = app().tickets().createNotifier(); | |||
notifier.sendMailing(ticket); | |||
redirect(TicketsPage.class, WicketUtils.newObjectParameter(getRepositoryModel().name, "" + ticket.number)); | |||
redirectTo(TicketsPage.class, WicketUtils.newObjectParameter(getRepositoryModel().name, "" + ticket.number)); | |||
} else { | |||
// TODO error | |||
} |
@@ -280,12 +280,12 @@ public abstract class RootPage extends BasePage { | |||
PageParameters params = getPageParameters(); | |||
if (params == null) { | |||
// redirect to this page | |||
manualRedirect(getClass()); | |||
redirectTo(getClass()); | |||
} else { | |||
// Strip username and password and redirect to this page | |||
params.remove("username"); | |||
params.remove("password"); | |||
redirect(getClass(), params); | |||
redirectTo(getClass(), params); | |||
} | |||
} | |||
} |