I have a porblem with popups. Sorry for my bad English, I hope they understand what I want.

/ **
* Integration of hook_popups () of the popups module.
* /
privatemsg_popups function () (
if (variable_get ( 'privatemsg_popups', TRUE)) (
return array (
'*' => Array (
'a [href *= messages / new'] / / Write new message
)
'*' => Array (
'div # block-menu-privatemsg-privatemsg a [href *= messages / new]',
)
);
)
)

When I activate the block on messages it goes with popups, but this only because I want somewhere to activate the road is probably 'a [href *= messages / new]' wrong so I need to change it.

Popups should be applied for 1 + 2 to 3 works. Here the link of the messages / new is correct.
When 1 + 2 is the link messages/new/1 / and messages/new/6 *** / ***
How can I change the path in the modules, the popups will work on 1 +2. Regards

CommentFileSizeAuthor
#3 privatemsg.popups_fix.patch1.77 KBberdir
#1 nachrichten.jpg62.02 KBfugazi

Comments

fugazi’s picture

StatusFileSize
new62.02 KB

Sorry I forgot the picture

fugazi’s picture

someone has a tip. I was very happy

berdir’s picture

Category: support » task
Status: Active » Needs review
StatusFileSize
new1.77 KB

You need to enable the "Scan all pages for popup links." option at admin/settings/popups.

Not idea what I've commited there, the second definition is not needed, atleast not with the above option set. The attached removes that and also adds a note about the required setting.

Status: Needs review » Needs work

The last submitted patch, privatemsg.popups_fix.patch, failed testing.

berdir’s picture

Status: Needs work » Needs review

#3: privatemsg.popups_fix.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, privatemsg.popups_fix.patch, failed testing.

berdir’s picture

Status: Needs work » Needs review

D'oh. See #719820: randomString() can cause invalid xpath in the tests why this failed twice in a row.

berdir’s picture

#3: privatemsg.popups_fix.patch queued for re-testing.

fugazi’s picture

Oh this is wonderful, it works. Many, many thanks. Fantastic.

berdir’s picture

Status: Needs review » Fixed

Fixed in 6.x-2.x-dev

Status: Fixed » Closed (fixed)

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