Specials by Category and Manufacturer need price range and messages

Specials by Category and Manufacturer need price range and messages
Include updates to products_price_sorter only when Special actually is changed
<a href='#crh-start'></a><a href='#crh-data-%7B%22processed%22%3A%20%5B%22https%3A//github.com/zencart/zc-v1-series/pull/145%23issuecomment-55794393%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23discussion_r17628282%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23discussion_r17628347%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23issuecomment-56075965%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23issuecomment-56077770%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23issuecomment-56078335%22%2C%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23issuecomment-56078537%22%5D%2C%20%22comments%22%3A%20%7B%22General%20Comment%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23issuecomment-55794393%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22%3A%2B1%3A%20%22%2C%20%22created_at%22%3A%20%222014-09-16T18%3A53%3A16Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/404472%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/drbyte%22%7D%7D%2C%20%7B%22body%22%3A%20%22Linda%2C%20I%20think%20%40texdc%20was%20referring%20to%20these%20lines%5Cr%5Cn%5Cr%5Cn%20%20%20%20%20%2B//%40%40TODO%20-%20remove%20%24new_special%20not%20used%3F%5Cr%5Cn%20%20%20%20%20%2B//%20%24new_special%20%3D%20%24db-%3EExecute%28%5C%22select%20specials_id%20from%20%5C%22%20.%20TABLE_SPECIALS%20.%20%5C%22%20where%20%20%20products_id%3D%27%5C%22%20.%20%28int%29%24products_id%20.%20%5C%22%27%5C%22%29%3B%5Cr%5Cn%5Cr%5CnIf%20you%20have%20a%20scenario%20where%20you%20think%20we%20may%20need%20to%20reference%20this%20line%20in%20the%20future%2C%20then%20maybe%20raise%20an%20issue%20referencing%20the%20lines.%22%2C%20%22created_at%22%3A%20%222014-09-18T17%3A46%3A27Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/227354%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/zcwilt%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%20am%20leaving%20in%20the%20%40%40TODO%20as%20at%20the%20time%20of%20testing%20how%20a%20bazillion%20ways%20this%20can%20be%20used%20there%20was%20a%20reason%20that%20might%20need%20to%20be%20reviewed%20for%20inclusion%20and%20at%20the%20moment%20it%20escapes%20me.%20It%20isn%27t%20gaining%20enough%20significant%20value%20to%20remove%20those%20at%20this%20time.%22%2C%20%22created_at%22%3A%20%222014-09-18T17%3A59%3A31Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1992427%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/ajeh%22%7D%7D%2C%20%7B%22body%22%3A%20%22I%20defer%20to%20your%20domain%20knowledge%20regarding%20pricing%20%3A%29%20%5Cr%5Cnagreed%20a%20line%20here%20or%20there%20isn%27t%20a%20problem.%5Cr%5Cn%3A%2B1%3A%20%22%2C%20%22created_at%22%3A%20%222014-09-18T18%3A03%3A30Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/227354%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/zcwilt%22%7D%7D%2C%20%7B%22body%22%3A%20%22It%27s%20about%20code%20cleanliness.%20%20If%20they%27re%20not%20needed%2C%20there%27s%20no%20need%20to%20keep%20them.%20%20If%20removed%20and%20needed%20later%2C%20it%27s%20easy%20to%20restore.%22%2C%20%22created_at%22%3A%20%222014-09-18T18%3A05%3A08Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1590605%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/texdc%22%7D%7D%5D%2C%20%22title%22%3A%20%22General%20Comment%22%7D%2C%20%22Pull%20b6b0c7ca2f4fdad33b2969d8ac926acc17129caa%20admin/includes/languages/english/specials.php%2022%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23discussion_r17628282%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22Needs%20final%20newline.%22%2C%20%22created_at%22%3A%20%222014-09-16T20%3A29%3A56Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1590605%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/texdc%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20admin/includes/languages/english/specials.php%3AL91-102%22%7D%2C%20%22Pull%20b6b0c7ca2f4fdad33b2969d8ac926acc17129caa%20admin/specials.php%2039%22%3A%20%7B%22html_url%22%3A%20%22https%3A//github.com/zencart/zc-v1-series/pull/145%23discussion_r17628347%22%2C%20%22comments%22%3A%20%5B%7B%22body%22%3A%20%22I%27d%20go%20ahead%20and%20remove%20this%20%28and%20the%20other%20below%29.%20%20Version%20control%20will%20allow%20us%20to%20get%20it%20back%20if%20we%20need%20it%20and%20commented%20code%20like%20this%20tends%20to%20clutter%20things%20up.%22%2C%20%22created_at%22%3A%20%222014-09-16T20%3A30%3A58Z%22%2C%20%22user%22%3A%20%7B%22avatar_url%22%3A%20%22https%3A//avatars.githubusercontent.com/u/1590605%3Fv%3D2%22%2C%20%22html_url%22%3A%20%22https%3A//github.com/texdc%22%7D%7D%5D%2C%20%22title%22%3A%20%22File%3A%20admin/specials.php%3AL142-161%22%7D%7D%7D'></a>
<a href='https://www.codereviewhub.com/'><img src='http://www.codereviewhub.com/site/github-bar.png' height=40></a>
- [ ] <a href='#crh-comment-General Comment'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/zencart/zc-v1-series/pull/145#issuecomment-55794393'>General Comment</a></b>
- <a href='https://github.com/drbyte'><img border=0 src='https://avatars.githubusercontent.com/u/404472?v=2' height=16 width=16'></a> :+1:
- <a href='https://github.com/zcwilt'><img border=0 src='https://avatars.githubusercontent.com/u/227354?v=2' height=16 width=16'></a> Linda, I think @texdc was referring to these lines
+//@@TODO - remove $new_special not used?
+// $new_special = $db->Execute("select specials_id from " . TABLE_SPECIALS . " where   products_id='" . (int)$products_id . "'");
If you have a scenario where you think we may need to reference this line in the future, then maybe raise an issue referencing the lines.
- <a href='https://github.com/ajeh'><img border=0 src='https://avatars.githubusercontent.com/u/1992427?v=2' height=16 width=16'></a> I am leaving in the @@TODO as at the time of testing how a bazillion ways this can be used there was a reason that might need to be reviewed for inclusion and at the moment it escapes me. It isn't gaining enough significant value to remove those at this time.
- <a href='https://github.com/zcwilt'><img border=0 src='https://avatars.githubusercontent.com/u/227354?v=2' height=16 width=16'></a> I defer to your domain knowledge regarding pricing :)
agreed a line here or there isn't a problem.
:+1:
- <a href='https://github.com/texdc'><img border=0 src='https://avatars.githubusercontent.com/u/1590605?v=2' height=16 width=16'></a> It's about code cleanliness.  If they're not needed, there's no need to keep them.  If removed and needed later, it's easy to restore.
- [ ] <a href='#crh-comment-Pull b6b0c7ca2f4fdad33b2969d8ac926acc17129caa admin/includes/languages/english/specials.php 22'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/zencart/zc-v1-series/pull/145#discussion_r17628282'>File: admin/includes/languages/english/specials.php:L91-102</a></b>
- <a href='https://github.com/texdc'><img border=0 src='https://avatars.githubusercontent.com/u/1590605?v=2' height=16 width=16'></a> Needs final newline.
- [ ] <a href='#crh-comment-Pull b6b0c7ca2f4fdad33b2969d8ac926acc17129caa admin/specials.php 39'></a> <img src='http://www.codereviewhub.com/site/github-remaining.png' height=16 width=60>&nbsp;<b><a href='https://github.com/zencart/zc-v1-series/pull/145#discussion_r17628347'>File: admin/specials.php:L142-161</a></b>
- <a href='https://github.com/texdc'><img border=0 src='https://avatars.githubusercontent.com/u/1590605?v=2' height=16 width=16'></a> I'd go ahead and remove this (and the other below).  Version control will allow us to get it back if we need it and commented code like this tends to clutter things up.


<a href='https://www.codereviewhub.com/zencart/zc-v1-series/pull/145?mark_as_completed=1'><img src='http://www.codereviewhub.com/site/github-mark-as-completed.png' height=26></a>&nbsp;<a href='https://www.codereviewhub.com/zencart/zc-v1-series/pull/145?approve=1'><img src='http://www.codereviewhub.com/site/github-approve.png' height=26></a>&nbsp;<a href='https://github.com/zencart/zc-v1-series/pull/145'><img src='http://www.codereviewhub.com/site/github-refresh.png' height=26></a>
<a href='#crh-end'></a>