Kommentarer til: 113 dager er 3 måneder og 3 uker

Av: Eirik

Eirik — Mon, 14 Sep 2009 10:44:26 +0000

Det er en feil i koden her:


$segment_name = substr($key, 0, -1);

denne linjen fjerner flertalls s-en i weeks, months osv. I min versjon satt jeg den til å fjerne -er i minutter, måneder osv. Men: uker og timer fungerer ikke uten -er i entall, mens måned og sekund osv gjør det. Derfor har jeg lagt inn unntakshandtering av disse:


       if (($key == 'uker' && $value ==1)or($key == 'timer' && $value ==1)) {
            $segment_name = substr($key, 0, -1);
        } else {
        $segment_name = substr($key, 0, -2);
        }

I disse tilfellene fjernes kun -r, slik at det blir uke og time, og ikke "uk" og "tim". koden på snipplr er oppdatert.

Av: Eirik

Eirik — Mon, 31 Aug 2009 10:19:47 +0000

Det var mye off-topic :)
Tidsssone-tingen er rettet. Skal se om jeg finner en fin syntax-highlight sak til koden…

Takk for tilbakemeldingen!

Av: André

André — Mon, 31 Aug 2009 10:06:16 +0000

Off-topic: Tror også du må dobbeltsjekke tidssonen i WordPress for klokken er 12:05 hos meg nå (ikke 11:05).

Av: André

André — Mon, 31 Aug 2009 10:02:15 +0000

Først litt off-topic: Jeg tror du må velge en mørkere tekstfarge til kodeblokker, for hos meg er det nesten helt uleselig med lys tekst på lys bakgrunn.

On-topic: Når det gjelder behandling av tid i PHP, så er jeg ikke spesielt imponert over hvor mye (lite lesbar) kode som trengs for en så enkel funksjon… men det er uansett flott at du har gjort jobben enklere for oss andre som ikke jobber med PHP til daglig :)