-
12-10-2010, 15:53 #16Member
- Registered
- 17/07/02
- Location
- Wilrijk
- Posts
- 1,994
- iTrader
- 2 (100%)
- Mentioned
- 0 Post(s)
- Reputation
- 9/16
Dat bedoel ik dus met duidelijk geschreven codeMaar als ik dees zie dan vraag ik mij ook wel af wat het nut nog is.
//parse
$this->parse()
commentaar op zich is idd nodig, maar niet volgens een policy per functie of per class of *gasp* per regelAls ge OOP begint te programmeren op een project waar meerdere mensen aan werken en dat enorm veel klasses begint te omslaan dan is commentaar echt wel onontbeerlijk. Of ge nu megaduidelijke code schrijft of niet.
Het is vooral nodig bij workarounds + zaken die niet logisch lijken, of vrij moeilijke stukken dat het echt nodig is, voor interne libraries zie ik bv ook niet het nut voor elke functie van commentaar te voorzien omdan een documentje te genereren, liever dan een paar voorbeelden van hoe de library te gebruiken.no votes
-
-
12-10-2010, 17:15 #17Member
- Registered
- 23/07/07
- Location
- Gent
- Posts
- 25
- iTrader
- 1 (100%)
- Mentioned
- 0 Post(s)
Beter te veel commentaar dan te weinig. Ik ga akkoord met je parse voorbeeld, dit is inderdaad overbodig, maar als je de gewoonte hebt van alles te documenteren gaan moeilijkere stukken wél gedocumenteerd zijn. Je moet ergens een lijn trekken.
no votes
-
12-10-2010, 18:42 #18
Net wat we nodig hebben! Nog één... Voor persoonlijk wat aan te kloten geen problee he, aar koaan leer keer werken et de bestaande? Of zorgen jullie er zo voor dat klanten bij jullie vastzitten?
Raad eens welke letter kapot is...no votes
-
12-10-2010, 20:40 #19Member
- Registered
- 17/07/02
- Location
- Sol System
- Posts
- 10,064
- iTrader
- 1 (100%)
- Mentioned
- 0 Post(s)
- Reputation
- 27/78
Innovatie mag wel hoor, zeker als de bestaande op bepaalde gebieden schromelijk tekortschieten. Neem nu meertaligheid in Drupal, dat is gewoon de achilleshiel, en het is imho onbegrijpelijk dat daar niet meer aandacht aan besteed wordt - en al zeker voor een CMF dat zich als corporate oplossing wil opwerpen. Idem dito met display names trouwens, de eerste vraag daarnaar dateert van eind 2006 (Add a Display Name field to core in addition to Username | drupal.org), en nu - 4 jaar later - is daar eigenlijk - buiten wat halfslachtige, onvolledige dingen - nog steeds niks mee gebeurd, ondanks dat er enorm veel mensen naar vragen. Maar anderzijds heeft het ook wel zijn voordelen natuurlijk - zeker naar eindgebruikers toe kan je enorm veel met CCK, Views en Panels zonder 1 letter code te schrijven (performantie even buiten beschouwing gelaten
).
Los daarvan : Fork CMS zal het zeker niet worden voor mij, ik ben verknocht aan het Zend Framework ondertussen, en zal dus eerder voor een CMS opteren dat gebaseerd is op ZF (TomatoCMS zal ik wel eens bekijken denk ik).
Om terug te komen op de opmerkingen in verband met commentaar : nuttige commentaar die effectief iets bijbrengt is idd geen overbodige luxe, maar te veel nutteloze commentaar is ronduit idioot... Ik zou de tijd die je daaraan spendeert dan eerder in unit testing steken.
BTW Highcharts? Dat is toch niet gratis voor commercieel gebruik? Waar melden jullie dat, want dat is toch een vereiste volgens hun licentievoorwaarden?Last edited by dJeez; 12-10-2010 at 20:53.
PSN: dJeezBE - Delicious bookmarks
Disclaimer: I am currently suffering from severe CSD (Compulsive Sarcasm Disorder). - L'onion fait la farce - Facile largire de alienoPastafarian by choiceno votes
-
12-10-2010, 21:02 #20Crew Member
- Registered
- 01/09/02
- Location
- Peutie
- Posts
- 7,664
- iTrader
- 0
- Mentioned
- 4 Post(s)
- Reputation
- 13/105
Voor DisplayName kan ik je ergens volgen, al heb je dat met de Profile module snel opgelost.
En voor die meertaligheid, welke zijn de problemen die je ervaart? Ik heb een aantal meertalige websites die Drupal 6 draaien en ik heb daar toch niet echt problemen mee.Vanaf nu gaan we verder op BeyondGaming!
In deze thread wordt uitgelegd hoe je jouw account kan migreren.no votes
-
12-10-2010, 21:14 #21Member
- Registered
- 17/07/02
- Location
- Sol System
- Posts
- 10,064
- iTrader
- 1 (100%)
- Mentioned
- 0 Post(s)
- Reputation
- 27/78
De bedoeling is dat je een naam veld hebt dat overal verschijnt waar nu de login wordt gebruikt, en de login zelf enkel nog te gebruiken om effectief in te loggen (dus login is uniek, display name is dat niet - waar je gebruikers selecteert zou je wel beide gecombineerd te zien moeten krijgen zodat je de juiste persoon kan selecteren). Zonder de core aan te passen kan je dat bij mijn weten niet echt bekomen (theme_username dekt die lading niet)...
Het gaat over Drupal 7 (momenteel in beta, maar dus wel al feature freeze). Je gaat daar nog steeds i18n (of een gelijkaardige module) nodig hebben om ook maar iets degelijks te kunnen doen met meertaligheid. Dat is iets dat in core moet steken, en niet achteraf bijgeplakt moet worden (maar dat is voer voor een andere discussie - we moeten deze thread daar niet voor hijacken
).
Last edited by dJeez; 12-10-2010 at 21:23.
PSN: dJeezBE - Delicious bookmarks
Disclaimer: I am currently suffering from severe CSD (Compulsive Sarcasm Disorder). - L'onion fait la farce - Facile largire de alienoPastafarian by choiceno votes
-
13-10-2010, 00:25 #22
Ze stellen hun licentie wel open source voor. Highcharts mag ook gratis gebruikt worden. Aannemelijk is natuurlijk zelf de licentie te betalen als je het commercieel gebruikt (dit mag naar mijn mening wel wat nadrukkelijker vermeld worden)
Wat zit er zo nogal in van (eventueel) te betalen software?Last edited by adrianhates; 13-10-2010 at 12:02. Reason: typo
no votes
-
13-10-2010, 10:49 #23Member
- Registered
- 23/07/07
- Location
- Gent
- Posts
- 25
- iTrader
- 1 (100%)
- Mentioned
- 0 Post(s)
We hebben een overeenkomst met de maker. Onder elke chart moet er wel "Highcharts" staan.BTW Highcharts? Dat is toch niet gratis voor commercieel gebruik? Waar melden jullie dat, want dat is toch een vereiste volgens hun licentievoorwaarden?
Als je de file manager van TinyMCE erin wil steken ga je wel moeten betalen vrees ik. Dit zit niet standaard mee in de huidige versie.
We zijn zelf niet tevreden van deze file manager en plannen een alternatief. We hebben nu een aantal standaard modules (e.g. blog, search, tags, pages) en enkele funkier modules (Google Analytics en Campaign Monitor integratie).
Hoe meer sites we zelf bouwen met Fork CMS bij Netlash, hoe meer modules er gaan zijn. Een paar modules die in de vorige versie van Fork zaten maar nog niet opnieuw gemaakt zijn/geport zijn: poll module, fotoalbum module, webshop (met Ogone integratie), Twitter integratie, FAQ (...) - zo zijn er nog wel wat meer.
Niet elke module die we maken wordt noodzakelijk open source: we willen de kwaliteit van het basispakket hoog houden. Goede, generiek (in de goede zin van het woord) geschreven modules kunnen in de open source versie. Klantspecifieke modules niet.
Net zoals Linux, jQuery en Rails is er een "core team" dat de codekwaliteit hoog houdt. We aanvaarden niet zomaar iedereen zijn code om mee te shippen.
Development buiten het core team zal zich vooral op modulevlak afspelen. We hopen dat andere webbureau's ook op Fork CMS beginnen ontwikkelen en dat we zo modules kunnen uitwisselen.
We kijken er op dit moment naar om op een opener manier te ontwikkelen dan enkel een zip file ter download te stellen (bugtracker, open code repository, e.d.)
Sommige modules voorzien hooks om met elkaar te spreken (bvb. tags is gekoppeld aan pagina's, en aan de zoekmachine; de zoekmachine voorziet koppelingen zodat de inhoud van een custom module ook doorzocht kan worden). Deze "hooks" moeten consistent gebruikt worden of we krijgen een codesoep die je niet wil meemaken. Vandaar de kwaliteitscontrole.
We willen niet in de situatie komen waar Structure V2.122 enkel werkt als je Plugin 3.8x102 geïnstalleerd hebt op CMS v2020.2030.1782.no votes
-
14-10-2010, 13:47 #24Approved 9-lifer
- Registered
- 03/12/03
- Location
- Oostende
- Posts
- 2,599
- iTrader
- 78 (100%)
- Mentioned
- 0 Post(s)
- Reputation
- 7/10
in de documentatie: Fork CMS - developer documentation
Apache 2.0 width mod_rewrite enabled
width zal dus with moeten zijn
het zit waarschijnlijk zodanig in de vingers maar toch ;-)
Moest ik nu geen huis aan het verbouwen zijn ging ik het zeker eens bekijken. Het zal voor ergens na de zomer zijn denk ik
.
no votes
-
15-10-2010, 10:23 #25Member
- Registered
- 23/07/07
- Location
- Gent
- Posts
- 25
- iTrader
- 1 (100%)
- Mentioned
- 0 Post(s)
Dankje, typo opgelost!
Voor de geïnteresseerden, we schreven nog enkele duidende blogposts: Fork CMS in publieke beta - Blog - Netlash Webdesign en Q&A - Blog - Fork CMS
-W.no votes

