Go to main content

Textpattern CMS support forum

You are not logged in. Register | Login | Help

#31 2009-09-10 18:54:44

mrdale
Member
From: Walla Walla
Registered: 2004-11-19
Posts: 2,215
Website

Re: mem_postmaster - Postmaster Revamp

colak > I’ll sponsor the paging fix. Don’t worry.

Offline

#32 2009-09-11 01:30:50

Manfre
Plugin Author
From: North Carolina
Registered: 2004-05-22
Posts: 588
Website

Re: mem_postmaster - Postmaster Revamp

v1.0.3 is posted

  • ZCR zemScubscriberLists works again. <txp:zem_contact_secret name=“zemSubscriberLists” value=“List1,List2,List3” />
  • subscriber list search results now page.

Offline

#33 2009-09-11 05:50:50

colak
Admin
From: Cyprus
Registered: 2004-11-20
Posts: 9,387
Website GitHub Mastodon Twitter

Re: mem_postmaster - Postmaster Revamp

Manfre wrote:

  • ZCR zemScubscriberLists works again. <txp:zem_contact_secret name=“zemSubscriberLists” value=“List1,List2,List3” />

! works beautifully !

  • subscriber list search results now page.

So does this!

Michael this community does want you back:)


Yiannis
——————————
NeMe | hblack.art | EMAP | A Sea change | Toolkit of Care
I do my best editing after I click on the submit button.

Offline

#34 2009-09-11 13:14:42

Manfre
Plugin Author
From: North Carolina
Registered: 2004-05-22
Posts: 588
Website

Re: mem_postmaster - Postmaster Revamp

v1.0.4 posted

Closed a few more of the inherited security holes. This one relates to an XSS hole (thanks wet) and a few more SQL injection holes I noticed. This plugin still has many security holes that still need to be plugged. I will gradually clean up the code when I have free time.

Thanks colak.

Offline

#35 2009-09-11 17:15:50

mrdale
Member
From: Walla Walla
Registered: 2004-11-19
Posts: 2,215
Website

Re: mem_postmaster - Postmaster Revamp

upm_savenew compatability

Just a quick reminder that if you’re using upm_savenew and don’t want extra “Save New” buttons in the postmaster area of the write screen, apply the following fix

At about line 45 replace a line that looks similar to this: $('#page-article input[name="save"]').eq(0).

Offline

#36 2009-09-15 07:05:02

candyman
Member
From: Italy
Registered: 2006-08-08
Posts: 684

Re: mem_postmaster - Postmaster Revamp

It seems that the latest version of mem_postmaster causes conflicts with some others plugins option menu in the Extensions panel.

After activating the plugin (not the library), in my case (TXP 4.0.8), the

  • rss_admin_db_manager (DB backup menu)
  • rss_live archive
  • tru_tags
  • msd_google_map
  • msd_colorpicker

options page stopped working.
Hope this may help the author to find the problem.

Last edited by candyman (2009-09-15 07:09:34)

Offline

#37 2009-09-15 07:48:34

colak
Admin
From: Cyprus
Registered: 2004-11-20
Posts: 9,387
Website GitHub Mastodon Twitter

Re: mem_postmaster - Postmaster Revamp

candyman wrote:

It seems that the latest version of mem_postmaster causes conflicts with some others plugins option menu in the Extensions panel.

After activating the plugin (not the library), in my case (TXP 4.0.8), the

  • rss_admin_db_manager (DB backup menu)

confirmed.

Error in the page:

Fatal error: Cannot redeclare pref_func() (previously declared in /users/home/xxx/web/public/textpattern/lib/txplib_misc.php(594) : eval()'d code:901) in /users/home/xxx/web/public/textpattern/include/txp_prefs.php on line 244

Yiannis
——————————
NeMe | hblack.art | EMAP | A Sea change | Toolkit of Care
I do my best editing after I click on the submit button.

Offline

#38 2009-09-15 14:04:56

Manfre
Plugin Author
From: North Carolina
Registered: 2004-05-22
Posts: 588
Website

Re: mem_postmaster - Postmaster Revamp

@candyman, change the plugin load order so that mem_postmaster is after all of those other plugins.

Offline

#39 2009-09-15 15:01:41

colak
Admin
From: Cyprus
Registered: 2004-11-20
Posts: 9,387
Website GitHub Mastodon Twitter

Re: mem_postmaster - Postmaster Revamp

Hi Michael
I have changed the plugin’s load order to ‘9’ using plugin composer but the error does not go away.


Yiannis
——————————
NeMe | hblack.art | EMAP | A Sea change | Toolkit of Care
I do my best editing after I click on the submit button.

Offline

#40 2009-09-15 15:32:31

Manfre
Plugin Author
From: North Carolina
Registered: 2004-05-22
Posts: 588
Website

Re: mem_postmaster - Postmaster Revamp

v1.0.5
  • Renamed pref_func().

The problem was caused by many plugins needing to redefine a core function and the other plugins did not properly guard with a function_exists() check. You should contact the plugin authors to get them to properly guard their code.

Offline

#41 2009-09-15 22:35:21

truist
Plugin Author
From: Ohio, USA
Registered: 2004-11-09
Posts: 257
Website

Re: mem_postmaster - Postmaster Revamp

Manfre, someone (colak?) alerted me to this problem (with tru_tags), and per your comment, it looks like I should be protecting a redefine of pref_func() somehow. But tru_tags doesn’t redefine, or even call, pref_func() so I’m not sure what the issue is, or how to fix it. Can you clarify?

Offline

#42 2009-09-16 00:00:14

Gocom
Developer Emeritus
From: Helsinki, Finland
Registered: 2006-07-14
Posts: 4,533
Website

Re: mem_postmaster - Postmaster Revamp

truist wrote:

Manfre, someone (colak?) alerted me to this problem (with tru_tags), and per your comment, it looks like I should be protecting a redefine of pref_func() somehow. But tru_tags doesn’t redefine, or even call, pref_func() so I’m not sure what the issue is, or how to fix it. Can you clarify?

You are redeclaring pref_func(). Pref_func() is a core function, defined in txp_prefs.php. But, in your plugin you have a line:

include(txpath . '/include/txp_prefs.php');

Which will cause conflicts with any plugin which includes the same file on same page load. To fix this, you can possibly, either use function check, class or include_once.

Offline

#43 2009-09-16 00:08:08

truist
Plugin Author
From: Ohio, USA
Registered: 2004-11-09
Posts: 257
Website

Re: mem_postmaster - Postmaster Revamp

Ah, that makes sense. I will fix this in the next version of tru_tags. Thank you!

Offline

#44 2009-09-16 00:29:47

Manfre
Plugin Author
From: North Carolina
Registered: 2004-05-22
Posts: 588
Website

Re: mem_postmaster - Postmaster Revamp

You also don’t want to include it when $event == ‘prefs’, since for some reason the core didn’t feel like using the *_once functions in ./textpattern/index.php.

Offline

#45 2009-09-16 05:08:00

colak
Admin
From: Cyprus
Registered: 2004-11-20
Posts: 9,387
Website GitHub Mastodon Twitter

Re: mem_postmaster - Postmaster Revamp

michael, The update seemed to have worked and the error disappeared.


Yiannis
——————————
NeMe | hblack.art | EMAP | A Sea change | Toolkit of Care
I do my best editing after I click on the submit button.

Offline

Board footer

Powered by FluxBB