Skip to content
This repository was archived by the owner on Nov 22, 2021. It is now read-only.

Don't use hardcoded URLs #27

Merged
merged 3 commits into from
Oct 9, 2013

Conversation

aykevl
Copy link
Contributor

@aykevl aykevl commented Oct 4, 2013

Ik kan genoeg redenen bedenken waarom dit handiger is. Voor mij is het vooral handiger dat links gewoon blijven werken op een lokale installatie en niet naar karpenoktem.nl verwijzen. Verder helpt het met andere wijzigingen als toen we van http naar https gingen.

@bwesterb
Copy link
Member

bwesterb commented Oct 5, 2013

Je gebruikt hier en daar auri op een verkeerde manier. auri('wiki') gaat bijvoorbeeld niet altijd naar domein/wiki, maar soms ook naar domein/index.php/wiki afhankelijk van de instellingen.

Het is beter om misschien een euri toe te voegen met in config.php de links. Dus zoiets waar

euri($blaat) == $cfg['links'][$blaat]

@aykevl
Copy link
Contributor Author

aykevl commented Oct 5, 2013

Ik heb het op precies dezelfde manier gebruikt als hoe het ook in het menu staat:

                    <li><a href="<?php echo auri('wiki') ?>">Wiki</a></li>

Is het echt zo belangrijk om de wiki-link configureerbaar te maken? Ik zou denken dat die in principe niet zou moeten veranderen.

Wat betekenen auri, curi en euri eigenlijk?

@bwesterb
Copy link
Member

bwesterb commented Oct 6, 2013

Dat is dan een fout (van mij, lijkt het). euri bestaat nog niet. curi verwijst naar plaatjes. auri verwijst naar "actions". Ik zal wel een euri maken.

@bwesterb
Copy link
Member

bwesterb commented Oct 6, 2013

Zie c08d864

@aykevl
Copy link
Contributor Author

aykevl commented Oct 6, 2013

Bij deze. De wiki-link gebruikt euri.

@aykevl
Copy link
Contributor Author

aykevl commented Oct 9, 2013

Ik zag dat er een fout zit in https://karpenoktem.nl/links. Onder de link "onze eigen site" staat assertion failed.
De laatste commit hier lost die fout op.

Bij deze ook een berichtje dat er dus een commit is :)

bwesterb added a commit that referenced this pull request Oct 9, 2013
@bwesterb bwesterb merged commit b318f02 into karpenoktem:online Oct 9, 2013
@bwesterb
Copy link
Member

bwesterb commented Oct 9, 2013

Dank je.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants