| # |
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 |
| # |
Mar 12th 2018, 15:48 |
dorxy |
if you execute `$this->request->session()->write('User.id',$this->request->getParam('id'));` again on the next page, and maybe the ID param is not present in the URI at that point? |
| # |
Mar 12th 2018, 15:47 |
dorxy |
is the id parameter persisted between the pagination requests? |
| # |
Mar 12th 2018, 15:46 |
learningcake |
'63' being the parameter |
| # |
Mar 12th 2018, 15:46 |
learningcake |
http://localhost:8765/mystore/63 |
| # |
Mar 12th 2018, 15:46 |
dorxy |
@learningcake it also depends where exectly you execute the write and read |
| # |
Mar 12th 2018, 15:45 |
raul338 |
should be getQuery / getData ? |
| # |
Mar 12th 2018, 15:45 |
learningcake |
I created a method that searches database base on id parameter |
| # |
Mar 12th 2018, 15:43 |
raul338 |
why getParam('id') ? |
| # |
Mar 12th 2018, 15:43 |
learningcake |
any idea why my session doesn't work? |
| # |
Mar 12th 2018, 15:35 |
kitcat711 |
thx |