no, nothing seems weird indeed. i guess the next step is for you to set a breakpoing in crypted encoding strategy and see why your url is not being encrypted. if you still cant figure it out you are welcome to create a quickstart.
-igor On Tue, May 11, 2010 at 7:27 AM, Fernando Wermus <[email protected]> wrote: > I don't see anything weird in BookmarkablePageIncrementLink code, any help I > do appreciate > > package com.misPartidos.web.widgets.seo.navigation; > > import org.apache.wicket.Page; > import org.apache.wicket.PageParameters; > import org.apache.wicket.markup.html.link.BookmarkablePageLink; > import org.apache.wicket.markup.html.navigation.paging.IPageable; > > public class BookmarkablePageIncrementLink extends BookmarkablePageLink { > private static final long serialVersionUID = 5584517284841095213L; > > /** The PageableListView the page links are referring to. */ > protected final IPageable pageable; > �...@suppresswarnings("unchecked") > public BookmarkablePageIncrementLink(String id, IPageable pageable, Class > clazz, PageParameters p) { > super(id, clazz, p); > setAutoEnable(true); > this.pageable = pageable; > } > /** > * Determines the next page number for the pageable component. > * > * @return the new page number > */ > public final int getPageNumber() > { > // Determine the page number based on the current > // PageableListView page and the increment > int idx = pageable.getCurrentPage() + > getPageParameters().getInt("increment"); > > // make sure the index lies between 0 and the last page > return Math.max(0, Math.min(pageable.getPageCount() - 1, idx)); > } > > /** > * @return True if it is referring to the first page of the underlying > PageableListView. > */ > public boolean isFirst() > { > return pageable.getCurrentPage() <= 0; > } > > /** > * @return True if it is referring to the last page of the underlying > PageableListView. > */ > public boolean isLast() > { > return pageable.getCurrentPage() >= (pageable.getPageCount() - 1); > } > > /** > * Returns true if the page link links to the given page. > * > * @param page > * ignored > * @return True if this link links to the given page > * @see > org.apache.wicket.markup.html.link.PageLink#linksTo(org.apache.wicket.Page) > */ > public boolean linksTo(final Page page) > { > int increment=getPageParameters().getInt("increment"); > pageable.getCurrentPage(); > return ((increment < 0) && isFirst()) || ((increment > 0) && isLast()); > } > } > > > On Mon, May 10, 2010 at 5:33 PM, Igor Vaynberg <[email protected]>wrote: > >> indeed, looking at the code everything is encrypted, so the problem >> must be in your BookmarkablePageIncrementLink. only urls that are >> generated via the coding strategy are encrypted, if you concatenate >> strings yourself they will not be encrypted obviously. >> >> -igor >> >> On Mon, May 10, 2010 at 12:06 PM, Fernando Wermus >> <[email protected]> wrote: >> > Igor, >> > Wicket in Action explains >> > "... Using this code, Wicket will encrypt all your URLs—including >> > bookmarkable URLs. ...". >> > >> > I need fixed entry point for my stateless page, but not readable for >> > humans, because some hacker would like to extract all the information >> from >> > the site. >> > >> > How can I achieve this? >> > >> > >> > On Mon, May 10, 2010 at 4:01 PM, Igor Vaynberg <[email protected] >> >wrote: >> > >> >> afair crypted strategy only encodes non-bookmarkable urls. it does not >> >> encode bookmarkable urls because those are meant as entrypoints into >> >> your application. >> >> >> >> -igor >> >> >> >> On Mon, May 10, 2010 at 11:38 AM, Fernando Wermus >> >> <[email protected]> wrote: >> >> > Hi all, >> >> > I create a PagingNavigator stateless. Instead of using a model to >> have >> >> > the number page shown, my StatelessPagingNavigator shows the number >> >> through >> >> > parameters. I hope that the page number wouldnt have been showed using >> >> > CryptedUrlWebRequestCodingStrategy, but It does. This is rather weird; >> I >> >> > hope any could point me out some solution. Is my class written wrongly >> in >> >> > some way? >> >> > >> >> > >> >> > public class StatelessPagingNavigator extends PagingNavigator { >> >> > private static final long serialVersionUID = 3576836044400027436L; >> >> > >> >> > public StatelessPagingNavigator(String id, DataView dataView) { >> >> > super(id, dataView); >> >> > } >> >> > >> >> > �...@override >> >> > protected Link newPagingNavigationIncrementLink(final String id, >> >> IPageable >> >> > pageable, int increment) { >> >> > * **PageParameters p=new PageParameters();* >> >> > * >> >> > * >> >> > * **p.add("increment", String.valueOf(increment));* >> >> > * **p.add("pageNumber", String.valueOf(pageable.getCurrentPage()));* >> >> > Link link= new BookmarkablePageIncrementLink(id, pageable, >> >> > getPage().getClass(), p){ >> >> > private static final long serialVersionUID = 1L; >> >> > >> >> > public boolean isEnabled() >> >> > { >> >> > return super.isEnabled() && StatelessPagingNavigator.this.isEnabled() >> && >> >> > StatelessPagingNavigator.this.isEnableAllowed(); >> >> > } >> >> > }; >> >> > return link; >> >> > } >> >> > >> >> > @Override >> >> > protected Link newPagingNavigationLink(final String id, final >> IPageable >> >> > pageable, int pageNumber) { >> >> > * **PageParameters p=new PageParameters();* >> >> > * **p.add("pageNumber", String.valueOf(pageNumber));* >> >> > return new BookmarkablePagingNavigationLink(id, pageable, >> >> > getPage().getClass(), p){ >> >> > private static final long serialVersionUID = -3076648671049640420L; >> >> > >> >> > public boolean isEnabled() >> >> > { >> >> > //return false; >> >> > return super.isEnabled() && StatelessPagingNavigator.this.isEnabled() >> && >> >> > StatelessPagingNavigator.this.isEnableAllowed(); >> >> > } >> >> > }; >> >> > } >> >> > >> >> > @Override >> >> > protected PagingNavigation newNavigation(final IPageable pageable, >> final >> >> > IPagingLabelProvider labelProvider) { >> >> > return new PagingNavigation("navigation", pageable, labelProvider) { >> >> > private static final long serialVersionUID = 1102823179571300337L; >> >> > >> >> > @Override >> >> > protected Link newPagingNavigationLink(final String id, final >> IPageable >> >> > pageable, int pageIndex) { >> >> > * **PageParameters p=new PageParameters();* >> >> > * **p.add("pageIndex", String.valueOf(pageIndex));* >> >> > return new BookmarkablePagingNavigation(id, pageable, >> >> getPage().getClass(), >> >> > p){ >> >> > private static final long serialVersionUID = -3076648671049640420L; >> >> > >> >> > public boolean isEnabled() >> >> > { >> >> > return super.isEnabled() && StatelessPagingNavigator.this.isEnabled() >> && >> >> > StatelessPagingNavigator.this.isEnableAllowed(); >> >> > } >> >> > }; >> >> > } >> >> > }; >> >> > } >> >> > } >> >> > >> >> > thanks in advance >> >> > >> >> > -- >> >> > Fernando Wermus. >> >> > >> >> > www.linkedin.com/in/fernandowermus >> >> > >> >> >> >> --------------------------------------------------------------------- >> >> To unsubscribe, e-mail: [email protected] >> >> For additional commands, e-mail: [email protected] >> >> >> >> >> > >> > >> > -- >> > Fernando Wermus. >> > >> > www.linkedin.com/in/fernandowermus >> > >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> >> > > > -- > Fernando Wermus. > > www.linkedin.com/in/fernandowermus > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
