Zikula Community

• pnFilip
  
  

  • Rank: Softmore
  • Registered: Jan 24, 2006
  • Last visit: Apr 19, 2010
  • Posts: 150

  Posted: Mar 30, 2009 - 09:27 PM

  Hello,
  I am trying to use Pagelock with a return address set like in the documentation.
  

  Code

  $returnUrl = pnModUrl('yourmodule', 'user', 'view');
  
  pnModAPIFunc('PageLock', 'user', 'pageLock',
                array('lockName' => "yourmodule{$mid}",
                      'returnUrl' => $returnUrl));

  
  But when a page is locked and I click on the "Back" button, the return address is not written correctly. The function converts special chars into entities name, in example it gives me the address : http://yoursite.com/index.php?module=yourmodule&type=user&func=view
  
  So is it a bug ? or something I do wrong ?
  Thanks for your help,
  Philippe

  • 
  •  
• JørnWildt
  
  

  • Rank: Softmore
  • Registered: Jul 02, 2002
  • Last visit: Oct 21, 2009
  • Posts: 240

  Posted: Mar 30, 2009 - 11:06 PM

  The pnRender plugin version of pnModURL does that encoding for use in templates (it must be encoded in tags in order to validate). Maybe the core pnModUrl does so too - I don't exactly know where the encoding takes place.

  • 
  •  
• Guite
  
  

  • Rank: Team Member
  • Registered: Jan 19, 2003
  • Last visit: Apr 06, 2010
  • Posts: 442

  Posted: Mar 30, 2009 - 11:27 PM

  Please try to change line 23 of system/PageLock/pnuserapi.php
  from
  

  Code

  $returnUrl = (array_key_exists('returnUrl', $args) ? htmlspecialchars($args['returnUrl']) : null);

  
  to
  

  Code

  $returnUrl = (array_key_exists('returnUrl', $args) ? $args['returnUrl'] : null);

  
  
  --
  Guite | ModuleStudio

  • 
  •  
• pnFilip
  
  

  • Rank: Softmore
  • Registered: Jan 24, 2006
  • Last visit: Apr 19, 2010
  • Posts: 150

  Posted: Mar 30, 2009 - 11:59 PM

  OK, it works great without "htmlspecialchars".
  Thanks again,
  Philippe

  • 
  •  
• Guite
  
  

  • Rank: Team Member
  • Registered: Jan 19, 2003
  • Last visit: Apr 06, 2010
  • Posts: 442

  Posted: Mar 31, 2009 - 12:06 AM

  I created ticket #1012 for reference.
  
  --
  Guite | ModuleStudio

  • 
  •  
•  
•  

Users on-line

• 0 users

This list is based on users active over the last 60 minutes.