Possible Enhancement to Newsletter List View

More
1 year 5 months ago - 1 year 5 months ago #1 by Chris
There is a menu option available for a listing of the available Newsletters.   Currently, the only NSPro options available are 1) Newsletter Order and 2) Mailing List ID.  It maybe helpful to all to provide a LIMIT option to limit the number of rows retrieved.  In our case, we send out 1-2 emails a week during our baseball season; however, i don't want to display 50-100 newsletters to the user.  I would prefer to be able to limit the number displayed.  This could be done via a set number or possibly something date based (e.g. don't show newsletters that are 'n' days old).

In the interim, I made the following code change (line 60 - following the setOrder statement) to "com_nspro/views/nspro/tmpl/newsletterlist.php" to limit the number of rows returned in the view:

$query->setLimit(25);

Additional changes would be required in the corresponding XML file to make this more configurable.

Thanks again Christopher for a great extension.

-Chris
 
Last edit: 1 year 5 months ago by Chris.

Please Log in or Create an account to join the conversation.

More
1 year 5 months ago - 1 year 5 months ago #2 by Chris
I went ahead and made changes to the XML and the PHP on our site to limit the display so you have a reference. The source code (XML and PHP) are attached in the zip file. -Chris
Last edit: 1 year 5 months ago by Chris.

Please Log in or Create an account to join the conversation.

  • Christopher Mavros
  • Christopher Mavros's Avatar
  • Offline
  • Administrator
  • Administrator
More
1 year 3 months ago #3 by Christopher Mavros
Replied by Christopher Mavros on topic Possible Enhancement to Newsletter List View
Hello Chris and thank you for your contribution.
I just included your changes in the main version.
They will be available in the next release.

Christopher Mavros
me@mavxr.com
If you like our extensions, please rate us on the JED!

Please Log in or Create an account to join the conversation.

More
1 year 3 months ago #4 by Chris
thank you! So glad I could make a small contribution to an incredible extension.
-Chris

Please Log in or Create an account to join the conversation.

  • Christopher Mavros
  • Christopher Mavros's Avatar
  • Offline
  • Administrator
  • Administrator
More
1 year 3 months ago #5 by Christopher Mavros
Replied by Christopher Mavros on topic Possible Enhancement to Newsletter List View
Thanks to you!
You are more than welcome to make more suggestions and/or contributions!

Christopher Mavros
me@mavxr.com
If you like our extensions, please rate us on the JED!

Please Log in or Create an account to join the conversation.

Moderators: Christopher Mavros