Log message #4124279

# At Username Text
# Mar 12th 2018, 15:59 neon1024 Would generate links like `example.com/examples/foo/bar?page=3` for example. Depending on your routing config obviously
# Mar 12th 2018, 15:59 raul338 but the paginator keeps query data and url structrure
# Mar 12th 2018, 15:58 neon1024 The paginator has an option of url, which will form the base of the url being built. So you can pass in your url with the params you need to include in the pagination. `$this->Paginator->options(['url' => ['controller' => 'Examples', 'action' => 'index', 'foo' => 'bar']])`
# Mar 12th 2018, 15:58 learningcake raul338, not totally hide it if I can get it to work showing the id in the pagination. I just did a search method that makes use of GET and pagination works fine
# Mar 12th 2018, 15:58 dereuromark the bottom line is: a filter as session is a usability killer. instead you should have good URLs by default.
# Mar 12th 2018, 15:57 dereuromark dorxy: the post is not necessary if it doesnt come from a form - as a normal link it is fine as get. thats part of prg.
# Mar 12th 2018, 15:57 neon1024 https://github.com/cakephp/cakephp/blob/master/src/View/Helper/PaginatorHelper.php#L53
# Mar 12th 2018, 15:56 learningcake @neon1024, can you explain more what you meant by pass the url into the helper
# Mar 12th 2018, 15:56 raul338 i don't understand why learningcake want to hide the store id in the pagination
# Mar 12th 2018, 15:56 neon1024 It prevents resubmission
# Mar 12th 2018, 15:55 learningcake was thinking session
# Mar 12th 2018, 15:55 dorxy why post pagination?
# Mar 12th 2018, 15:55 dorxy although I don't completely understand how that would be applicable here
# Mar 12th 2018, 15:55 learningcake hmm. thank you Geniuses
# Mar 12th 2018, 15:55 dorxy TIL :slightly_smiling_face: thanks
# Mar 12th 2018, 15:54 neon1024 Totally agree!
# Mar 12th 2018, 15:54 dereuromark neon1024: the idea is the same, you can just use that and do it on your own using that idea, but session is never the right solution here :slightly_smiling_face:
# Mar 12th 2018, 15:54 neon1024 Post Redirect Get, https://en.wikipedia.org/wiki/Post/Redirect/Get
# Mar 12th 2018, 15:53 dereuromark dorxy: https://github.com/FriendsOfCake/search#component
# Mar 12th 2018, 15:53 neon1024 Well, although I agree, it’s a bit heavy for what’s needed. Personally I would pass the url into the helper
# Mar 12th 2018, 15:53 dorxy @dereuromark what is prg?
# Mar 12th 2018, 15:53 learningcake okay
# Mar 12th 2018, 15:53 dereuromark I just told you: query string, prg, search plugin
# Mar 12th 2018, 15:52 learningcake please, how do I do it the right way,
# Mar 12th 2018, 15:52 dereuromark do it the right way, hacking it here only brings in bugs and usability issues
# Mar 12th 2018, 15:52 dereuromark he should look into prg pattern and foc/search plugin instead
# Mar 12th 2018, 15:52 learningcake yes
# Mar 12th 2018, 15:52 raul338 in that case the pagination helper should keep the id
# Mar 12th 2018, 15:51 raul338 I think learningcake is trying to make a pagination inside a view, like /stores/63?page=2
# Mar 12th 2018, 15:51 dorxy is the reason you're writing it to the session because it is not persisted between pagination? or because you wish to access it for other requests as well?
# Mar 12th 2018, 15:51 neon1024 Then might I recommend using a url friend slug of the username?
# Mar 12th 2018, 15:51 learningcake @neon, it is meant to be a public data. thank you
# Mar 12th 2018, 15:50 dereuromark I recommend using query strings instead as we probably all do :slightly_smiling_face:
# Mar 12th 2018, 15:50 neon1024 Well if you’re using incremental id’s and you are not checking the id, I could just change it to 64 and see another users data
# Mar 12th 2018, 15:50 dereuromark using the session for pagination stuff is usually a huge anti pattern and usability killer
# Mar 12th 2018, 15:50 dorxy I'm still wondering how/why exactly you're doing it this way
# Mar 12th 2018, 15:50 learningcake @neon, the id is exposed to the url. Any security risk?
# Mar 12th 2018, 15:50 dorxy perhaps you should put an if statement around your write function to determine if there even is an ID parameter?
# Mar 12th 2018, 15:49 neon1024 You can change the pagination url in the helper options
# Mar 12th 2018, 15:49 learningcake no, the id is not persisted between pagination requests. Don't know how to make it persistent since the url is meant to be a unique link http://localhost:8765/mystore/63
# Mar 12th 2018, 15:49 neon1024 I hope the user id isn’t exposed to the url