Geen lege albumnaam weergeven als de human name niet is ingevuld

karpenoktem / old-knfotos

Karpe Noktem fotoboek

http://karpenoktem.nl

3 stars 4 forks source link

Geen lege albumnaam weergeven als de human name niet is ingevuld #38

Closed Minnozz closed 7 years ago

Minnozz commented 11 years ago

In plaats daarvan de pathname gebruiken.

Niet getest, omdat ik geen test-setup heb.

aykevl commented 11 years ago

In install.txt staat heel kort hoe knfotos geïnstalleerd moet worden.

bwesterb commented 11 years ago

@Jille, @aykevl93 kan een van jullie kijken of dit ok is?

aykevl commented 11 years ago

Is dit een fix voor issue #16? @Minnozz Het zou handig zijn als je aanpassingen in een aparte branch doet. Voordat je begint: git checkout -b minozz-mijnaanpassing Dat maakt mergen een stuk makkelijker. Ik kijk er nu naar. Het lijkt in elk geval niks te breken (tot nu toe).

aykevl commented 11 years ago

Wat mij betreft een verbetering. Een paar puntjes:

$displayname... $albumtitle klinkt wat mij betreft beter
In templates/footer.tpl staat nog een $humanname (dit soort dingen zijn snel te vinden met git grep. Of is er een reden dat je deze niet verandert hebt?)

$humanname is nog wel nodig in templates/actions.tpl.

Jille commented 11 years ago

@aykevl93 Het maakt toch niks uit of hij het in een aparte branch doet of niet? master is ook gewoon een branch?

Code ziet er verder wel goed uit. Zodra ik tijd heb en de video-PR klaar is zal ik hem mergen.

Minnozz commented 11 years ago

@aykevl93: In de footer wordt inderdaad nog $pa['humanname'] gebruikt, met op die plaats een fallback: ?: $pa['name']. Het was inderdaad mooier geweest als die fallback in loadPathAlbums() had gezeten, maar dit doet hetzelfde.

aykevl commented 11 years ago

@Jille Oke. Ik dacht dat dat gebruikelijk was.

@Minnozz Dat is waar. Ik had de fallback niet gezien.

bwesterb commented 11 years ago

@Jille ping