tnc-ba / strongTNC

BYOD TNC Database Management Tool
GNU Affero General Public License v3.0
0 stars 0 forks source link

Paging: Refactor producers into classes? #244

Open dbrgn opened 10 years ago

dbrgn commented 10 years ago

I had some concerns regarding the original implementation (009d6eb), especially because the client (browser) could control so much information (template path, list and stat producers that didn't even have to match). Seems better now but I still wonder if it wouldn't be better to define a base class with two methods that a paging producer implements instead of registering two functions each.

See https://github.com/tnc-ba/strongTNC/commit/2ad954722d5e0e522bbcf445d4b7c46473d94d13#commitcomment-6448151