I like this module (and decided to use version 2.x since I'm using SimpleNews 2.x too...)
...but couldn't use it as-is for my client. It isn't clear at all, what the different 'Regions' in the sort page are.

The best way I saw to fix this, was to make the amount of 'regions' configurable. Then if the amount is one, do not output the 'No Region' header or region dropdowns, in the sort screen.

I don't know if you think this is a good configuration option in the admin screen... but I just made a dropdown for 1 to 10 regions.

Oh, and I hope you'll forgive me for getting rid of your ereg() calls at the same time:
* ereg() is deprecated
* you're using it only to match substrings, which is unnecessary. strpos() does nicely.

CommentFileSizeAuthor
#1 1179598-1.patch8.24 KBroderik
scs-noregions.patch7.9 KBroderik

Comments

roderik’s picture

StatusFileSize
new8.24 KB

oops, there were still some mistakes in the tpl.php...

vuil’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)

Unsupported 6.x version. I close the issue as Closed (outdated).