NSPro newsletterlist link

More
7 months 1 week ago #13 by Craig
Replied by Craig on topic NSPro newsletterlist link
Hi Christopher,  further to this issue, I had a look at all the changes between 3.3.3044 and 3.3.3068 and I can see the changes to include the new option "newsletter_list_requires_menu_item" and I can see the change in components/com_nspro/views/nspro/view.html.php that throws the exception I was suffering from but I cannot see any other changes relating to this option that would change the "back to newsletter list" URL that is displayed at the bottom of a newsletter item viewed online, when launched from the view online link in the newsletter email.

Could it be that some file update was missing from the 3.3.3068 release? I have made a small edit to components/com_nspro/views/nspro/tmpl/newsletterview.php that does replace the link with the menuitem
The following lines were inserted after line 127
Code:
  if (empty($itemid) && $this->cparams->get('params.newsletter_list_requires_menu_item', '0'))   {     $item_id = $db->setQuery('SELECT `id` FROM `#__menu` WHERE `link` LIKE "'.$db->escape('index.php?option=com_nspro&view=nspro&layout=newsletterlist').'"')->loadResult();     $itemid = "&Itemid=".$item_id;   }

 
The following user(s) said Thank You: Chris

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

  • Christopher Mavros
  • Christopher Mavros's Avatar
  • Offline
  • Administrator
  • Administrator
More
5 months 2 weeks ago #14 by Christopher Mavros
Replied by Christopher Mavros on topic NSPro newsletterlist link
Hi Craig!
Thank you for posting this.

I adjusted it a little bit and included it in the next version.

Christopher Mavros
me@mavxr.com
If you like our extensions, please rate us on the JED!
The following user(s) said Thank You: Craig

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

More
4 months 2 weeks ago #15 by Craig
Replied by Craig on topic NSPro newsletterlist link
Seems to be working fine in 3.4.3105, many thanks

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

Moderators: Christopher Mavros