1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Fixed Bug when manually inserting page number in URL of profile

Discussion in 'Resolved Bug Reports' started by imthebest, Apr 23, 2015.

  1. imthebest

    imthebest Formerly Super120

  2. Amaury

    Amaury Well-Known Member

    That's because Mike only has 14 pages of profile posts.
  3. imthebest

    imthebest Formerly Super120

  4. Daniel Hood

    Daniel Hood Well-Known Member

    I think either way of handling it is fine. If users are intentionally trying to access pages they shouldn't, I don't see the benefit of redirecting them somewhere we -think- they want to be.
    Amaury likes this.
  5. imthebest

    imthebest Formerly Super120

    Well if it isn't a bug then at least there is inconsistency between XF and XFMG.

    But I think that XFMG way of dealing with this is better. But in case you want to stick with the XF way then instead of "There are no messages on Mike's profile yet" (which isn't true) you should display something like "The page number you have requested doesn't exists".
  6. Daniel Hood

    Daniel Hood Well-Known Member

    Or... you can ban the members trying to do malicious things on your site because there's very few people that manually change a part of the url like that without bad intent.

    I don't mind if they change the behavior. I guess it's cool Chris did in the gallery. But this is really a non issue because it doesn't really affect anything. It'd be an issue if there the pager buttons linked there but nothing links there. You have to manually tamper with the url in order to even see that message.
    Amaury likes this.
  7. imthebest

    imthebest Formerly Super120

    Well not only on XFMG but also on XF threads: https://xenforo.com/community/threads/searchaction-markup-for-google.83223/page-20

    Actually the RM behaves a bit different and just redirects you to the Overview (when viewing reviews): https://xenforo.com/community/resources/s9e-media-bbcodes-pack.2476/reviews?page=20

    And a blank page (when viewing updates): https://xenforo.com/community/resources/s9e-media-bbcodes-pack.2476/updates?page=20

    Not something critical I agree but at least it's clear we need some consistency here.
  8. Daniel Hood

    Daniel Hood Well-Known Member

    We'll agree to disagree. To me this couldn't be less of an issue. These are urls that nobody should ever visit. Anyone doing that is just looking for bugs to exploit (not literally -anyone- but definitely the strong majority, the rest are trying to shortcut to a page they recall and that's super rare).
    Amaury likes this.
  9. Mike

    Mike XenForo Developer Staff Member

    Fixed, thanks.
    Liam W and imthebest like this.
  10. Valhalla

    Valhalla Well-Known Member

    In many cases you get taken to the last page available.

Share This Page