When the PayPal block is enabled, the page title (node) is substituted for the subscription name.

Happens only when user is logged in. Does not happen with anonymous user

I am using a slightly modified 'interactive_media' theme and it occurred to me that I may have damaged my page.tpl.php file giving rise to the error. Therefore I have checked the situation using the standard Garland theme and can report the same error.

Screenshots:
http://www.mrpauldriver.com/images/Screenshot_1.jpg
http://www.mrpauldriver.com/images/Screenshot_2.jpg

CommentFileSizeAuthor
#12 lm_paypal_subscription-option_to_set_page_title_or_not.patch3.71 KBfleclerc

Comments

mrpauldriver’s picture

mrpauldriver commented
Priority: Normal » Critical
lyricnz’s picture

lyricnz commented
Priority: Critical » Normal

This is not critical

mrpauldriver’s picture

mrpauldriver commented

Apologies for misunderstanding of protocol, I'm new to this.

Are you able to confirm as a bug or error on my part.

lyricnz’s picture

lyricnz commented

It seems likely you are right - I vaguely recall some code that sets the page title when it probably shouldn't. Which LM Paypal block are you talking about? (PS: none of the module maintainers are very active right now - we are relying on the community to provide patches for any reported defects)

mrpauldriver’s picture

mrpauldriver commented

It is the standard block that is bundled with the module.

Screenshot: http://www.mrpauldriver.com/images/Screenshot_4.png

Note also that some tags appear in the block name

lyricnz’s picture

lyricnz commented
Status: Active » Fixed

lm_paypal consists of several modules - both the donations module, and the subscriptions module, provide blocks.

I've applied an (ugly) fix to 6.x-2.x-dev (wait until the tgz is rebuilt if you're not using cvs). http://drupal.org/project/cvs/88100

mrpauldriver’s picture

mrpauldriver commented

Thank you. I'll look out for the tgz file

lyricnz’s picture

lyricnz commented

It was there well before you posted - http://ftp.drupal.org/files/projects/lm_paypal-6.x-2.x-dev.tar.gz

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

dan_metille’s picture

dan_metille commented
Version: 6.x-2.x-dev » 7.x-1.x-dev
Component: Code » LM Paypal Subscriptions

This issue seems to happen again with the D7 version and the LM Paypal Subscriptions module.

Maybe the D6 fixe can be updated?

dan_metille’s picture

dan_metille commented
Status: Closed (fixed) » Active

I have commented out the line 73 of the 'lm_paypal_subscriptions.pages.inc' file:

drupal_set_title(t('Subscription @name', array('@name' => $so->item_name)), PASS_THROUGH);

and the problem disappears... but I'm doubting that this should be the right solution ;)

fleclerc’s picture

fleclerc commented
Status: Active » Patch (to be ported)
StatusFileSize
new lm_paypal_subscription-option_to_set_page_title_or_not.patch3.71 KB

Patch attached.

adding an option in /admin/config/lm_paypal/subscriptions_settings to set the title or not.

thanks