Vlákno názorů k článku Ministerstvo financí dalo zdrojáky Supervizora na GitHub od mno - Je to takove cele PHPkove, ale kdyby tam...

Je to takove cele PHPkove, ale kdyby tam bylo to CSS nekde pohromade a nacitalo se to aspon do iframu v indexu kdyz uz ne javascriptem jako SPA tak me to ani prilis neurazi.

Myslim ze pan by klidne mohl delat senior PHP architekta v nejake webove agosce.

ccs v iframu nedává smysl a ani by nefungovalo. SPA je jen buzzworld a zbytečnost.

Ne nemohl, kód je přišernej a neodpovídá úrovni, která se klade na PHP programátory, seniorní úroveň tomu je vzdálená na míle.

Je to ale hezké gesto a pokud se nebudou bránit PR, rád něco přišpějii, od toho je komunita.

Osobně bych přispěl max přepsáním celýho kódu. Tady nic lepšího nepomůže.

Kód je zajímavý, ale trochu (moc) nepřehledný. Vysvětlím proč

Celá webappka je ukázkou neudržovatelného, "špagetového" kódu. Nikde náznak jakékoliv architektury, prakticky neexistuje nějaká objektová architektura, není oddělena prezentační vrstva a datový model, ... prrostě neodpovídá úrovni požadované současným businessem.

Doporučuji celou aplikaci (pokud ji chcete dál rozšiřovat), přes nějaký PHP framework např. nette.org a trošku zapracovat na designu celé aplikace.

V tom případě OK, je fajn, že se tomu věnujete.

Bohužel jak se píše výše, tak je to spíše na komplet přepis a absence jakýchkoli rozumných komentářů (typu "tohle má dělat tohle" ne "regional settings" před třemi řádky nastavení locales) taky orientaci moc neusnadňuje.

podle linkedin.com/in/mar­tinkopecek máte velké zkušenosti, takže nechápu.

ten senior architekt byl sarkasmus, takhle nejak si predstavuju cloveka co jeste dneska dela PHP.

css ve vlastnim souboru a vlozene jak do indexu tak do iframe pagi by fungovalo.

Ačkoliv v PHP nevyvíjím, kód je pro mě těžko čitelný. Většinou jsem schopen pochopit dobře napsaný kód i v jiných jazycích, než v těch, které jsem byl schopen poznat.

Tento mi moc čitelný nepřijde.