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

Use <?php instead of <? in utils/install #26

Open
wants to merge 1 commit into
base: online
Choose a base branch
from

Conversation

aykevl
Copy link
Contributor

@aykevl aykevl commented Oct 4, 2013

This is necessary in the default config of Debian testing
(LMDE ~ october 2013)

This is necessary in the default config of Debian testing
(LMDE ~ october 2013)
@bwesterb
Copy link
Member

bwesterb commented Oct 5, 2013

Hier en daar gebruiken we ook <?=. Dan zou <? toch ook moeten werken?

@aykevl
Copy link
Contributor Author

aykevl commented Oct 5, 2013

Dat had ik ook verwacht, maar blijkbaar werkt <?= wel en <? niet.

Ik kan <?= ook wel in <?php echo veranderen als dat beter is.

@bwesterb
Copy link
Member

bwesterb commented Oct 5, 2013

Dit is toch gewoon een php.ini setting? Ik gebruik liever wel <?= ipv <?php echo voor de leesbaarheid.

@aykevl
Copy link
Contributor Author

aykevl commented Oct 5, 2013

Dat lijkt me ook. Ik denk dat het in de nieuwe Debian is veranderd. Aangezien het sowieso al een best-practice is om <?php te gebruiken en het bijna overal al wordt gebruikt heb ik dat hier ook veranderd.
(Ik ben er bijvoorbeeld een keer achter gekomen dat <?php toch wel erg handig is als je ook ergens <?xml in je code hebt staan. Dat was in een ongerelateerd project).

<?= is inderdaad wat leesbaarder.

@bwesterb
Copy link
Member

bwesterb commented Oct 5, 2013

@Jille Wat denk jij?

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