Fixes example and splits into another codeblock

Open umer936 opened this issue 3 years ago • 1 comments

Can probably have more clear wording on how passing the scope in paginate() works, but at the very least, the example is now functional like https://github.com/cakephp/docs/blob/4.x/en/controllers/pagination.rst is. The other page could also benefit from splitting the codeblock.

Jun 27 '22 23:06 umer936

I agree that I think it's wrong. Actually, when I added a 3rd Paginator, I ended up having to add the scope back into the parameters on the paginate() for some reason, so I'm not convinced that I've tested my proposed changes in this PR correctly.

But, since PaginatorComponent is being deprecated anyway, should I just make a PR with editing the current example so that it is functional (ie, just remove the word "Table")? And then handle a PR for splitting the codeblock into 2 examples after knowing it's explained correctly at a later date?

Jun 30 '22 15:06 umer936