Takk for tilbakemelding
1) Lisensinfo må jeg ta med ja, ordnes til neste utgave
2) OS-spesifikke filer... Jeg trodde jeg hadde fjernet dem, og når jeg sjekker innholdet i mappen med "ls -A verdata/" er det kun filene som skal være der som er der, men når jeg pakker den (
tar czvf verdata.tar.gz verdata/) så dukker alle disse ._ filene opp... eh, hva/hvor/når/hvordan unngår jeg det..?
3) Jah, flytte lokal cache til "files", ordnes til neste utgave
4)
verdata_cURLInit() er egentlig et lite lappeteppe av snippets fra den store verdensveven, så den er sikkert litt skakk. Men den funker jo
Jeg får se om jeg får tid til å sette meg mer inn i cURL og evt skrive den om
5) Kodelinjene kan jeg alltids legge inn linebreaks i
6) Kommentarer til loops lagt inn til neste utgave
7) Det med syntax ja, jeg har fått med meg at indentation skal være slik, men det med { } vs
if () : endif; er bare en vane jeg har lagt meg til. Jeg mener på å ha sett andre contrib moduler på drupal som også gjør det slik. Er det noen ytelsesforskjell på dem? (ikke det at jeg tror det har noe å si for et så lite script som jeg har laget, men jeg ser jo at det evt hadde blitt et problem om alle modulforfattere hadde valgt den trege løsningen...
) Eller er det mer bare hvordan man liker å ha det?
Igjen, takk for tilbakemelding
Nå er det skoletid, men jeg skal legge ut en oppdatert versjon når jeg får tid til å gjøre ferdig endringene
edit:
Jeg sliter litt med bruken av drupals '
file_check_directory()'. Det er ikke strengt nødvendig, ettersom den som installerer modulen også kan opprette mappen, eller jeg kan skrive en slik funksjon selv, men – det er jo en penere løsning å la Drupal opprette det med de innebygde funksjonene