Http\Url::getUrl('/') gives domain plus index.php about parable

[devvoh]

That's exactly what I needed to know. 0.12.8 is building in develop as we speak.

So of course you, the weird one, place Parable outside of the document root. So the document root isn't actually part of the script name. So it can't find the homeDir value + index.php, so it doesn't strip it, leaving index.php.

Good bug find :P

[devvoh]

Can you share your main Config with the parable data? So obviously I'm not looking for your db credentials ;)

This has to do with your homeDir setup, and I'll fix this ASAP and add some tests to precent this in the future.

[jerry1970]

Oh wait, that might be a leftover from a previous version, I have not set homeDir in the config. My public_html (as assumed by the Apache config that I cannot change) is still a symlink to Parable's public.

What is the correct syntax?

[devvoh]

Check https://github.com/devvoh/parable/blob/master/structure/app/Config/App.php_struct for the default Config. Let me know if that works!

[jerry1970]

Config is now:

        $config['parable'] = [
            'app' => [
                'title' => 'Test Project',
                'homeDir' => 'public_html',
            ],

But this in a template file:

<br /><?= $this->url->getUrl('/') ?>

gives me this:

http://testproject.dev/index.php/ 

Any idea?

[devvoh]

What's the return value of $this->url->getBasePath()?

[jerry1970]

/public_html/index.php

[devvoh]

That's what it should be. Okay. Can you tell me what the return values are for the following 2?

$this->request->getScriptName()
$this->url->getBaseUrl()

[jerry1970]

Sure:

/index.php
http://testproject.dev/index.php 

[devvoh]

https://github.com/devvoh/parable/releases/tag/0.12.8

There ya go! Just composer update that mofo and you should be fine!

Let me know if it works :)

[jerry1970]

PHPStorm, right-click on composer.json, Composer > Update...

http://testproject.dev/

👍 😃

[jerry1970]

I forgot I was using it differently from what you intended (framework outside doc root)... ;)

[devvoh]

It's cool, this was a good find and a necessary bugfix. To support different home dirs, it's necessary to make sure everything else works with it too.

😃