Ticket #599 (closed bug: fixed)

Opened 6 months ago

Last modified 6 months ago

Bug with the significant url engine when using urlhandler and noentrypoint

Reported by: mike Assigned to:
Priority: highest Milestone: Jelix 1.0.4
Component: jelix:plugins Version: 1.0.3
Severity: major Keywords: significant url
Cc: Php version:
Review: Hosting Provider:
Documentation needed: 0 Blocking:

Description

It seems there is a problem when using in combination an urlhandler class and the noentrypoint property from the urls.xml file.

For example in the url handler create() method, you have something like 

$url->pathInfo = '/test';

When you call jUrl::get(), you should have "/index.php/test"

But my results are:

  • if noentrypoint is false in your urls.xml, then jUrl::get() will return "/index.php/test" which is the correct result.
  • if noentrypoint is true, then jUrl::get() will return "//test" which isn't correct because your browser will interpret this as "http://test"

No doubts that this bug comes from the significant url plugin compilator...

Change History

05/21/08 14:55:18 changed by laurentj

  • status changed from new to closed.
  • resolution set to fixed.

In fact, in jUrl::toString didn't removed duplicated '/' between the scriptName and the pathInfo...

Fixed in the trunk and 1.0.x branch.

05/23/08 04:00:09 changed by bastnic

  • priority changed from high to highest.
  • status changed from closed to reopened.
  • resolution deleted.

I've no time to watch this problem, but it reveals the opposite for me.

Worked fine for we before this patch, all crashed now. Especially with form submit destination.

05/26/08 23:34:04 changed by laurentj

bastnic : please, explain your problem. "it doesn't work" says nothing to me.

05/27/08 00:19:24 changed by laurentj

  • status changed from reopened to closed.
  • resolution set to fixed.

ok, I found the bug. It's a bug in the form plugin. Added new methods jUrl::getPath() and jUrl::getQuery().

Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed