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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MrPaulDriver’s picture

MrPaulDriver CreditAttribution: MrPaulDriver commented
Priority: Normal » Critical
lyricnz’s picture

lyricnz CreditAttribution: lyricnz commented
Priority: Critical » Normal

This is not critical

MrPaulDriver’s picture

MrPaulDriver CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: MrPaulDriver commented

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

lyricnz’s picture

lyricnz CreditAttribution: 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.

sahaj’s picture

sahaj CreditAttribution: sahaj 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?

sahaj’s picture

sahaj CreditAttribution: sahaj 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 CreditAttribution: fleclerc commented
Status: Active » Patch (to be ported)
FileSize
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