abonnement Unibet Coolblue Bitvavo
  maandag 11 november 2013 @ 16:08:43 #121
125913 Devolution
Beep beep Richie
pi_133150571
En je hebt gecheckt dat in de config table van je website database jouw waarden staan (dat de default config table dus overschreven is)? En weet je ook zeker dat phpBB3 zijn configuratie-instellingen daar ook vandaan haalt?
"You know what Hell really is? It's not lakes of burning oil or chains of ice. It's being removed from God's sight."
pi_133152027
hij staat goed in mysql ja..

Toen ik hier op mijn lokale pc die tabel gekoppeld had aan een verse install (zelfde versie als de oude was.. nou weet ik dit niet zeker maar volgens mij liet ie het toen ook niet zien totdat ik em geupdate had.. maarja nou is ie volledig up to date dus dat schiet niet op.
pi_133208622
--

[ Bericht 100% gewijzigd door Chandler op 13-11-2013 08:42:58 ]
The people who lost my respect will never get a capital letter for their name again.
Like trump...
  woensdag 13 november 2013 @ 13:43:20 #124
137776 boem-dikkie
Jedi Mind Baby!
pi_133215796
quote:
0s.gif Op maandag 11 november 2013 16:44 schreef Skunk-m het volgende:
hij staat goed in mysql ja..

Toen ik hier op mijn lokale pc die tabel gekoppeld had aan een verse install (zelfde versie als de oude was.. nou weet ik dit niet zeker maar volgens mij liet ie het toen ook niet zien totdat ik em geupdate had.. maarja nou is ie volledig up to date dus dat schiet niet op.
Heet de database ook hetzelfde?
Ik weet niks van Hindoes. Wel van Samoerai en andere dingen.
pi_133227761
Wat vinden jullie van me scrippie? :D

http://pastebin.com/g2xTLLtP
The people who lost my respect will never get a capital letter for their name again.
Like trump...
pi_133232000
quote:
6s.gif Op woensdag 13 november 2013 19:42 schreef Chandler het volgende:
Wat vinden jullie van me scrippie? :D

http://pastebin.com/g2xTLLtP
Onleesbaar.
pi_133233032
quote:
6s.gif Op woensdag 13 november 2013 19:42 schreef Chandler het volgende:
Wat vinden jullie van me scrippie? :D

http://pastebin.com/g2xTLLtP
Bagger
  woensdag 13 november 2013 @ 21:40:32 #128
178193 Juicyhil
Bekende FOK!ker
pi_133233133
quote:
6s.gif Op woensdag 13 november 2013 19:42 schreef Chandler het volgende:
Wat vinden jullie van me scrippie? :D

http://pastebin.com/g2xTLLtP
Lekker vatbaar voor SQL Injection
Op dinsdag 9 augustus 2011 23:01 schreef SuperrrTuxxx het volgende:
Ik hou zoveel van jou, ik doe alles voor je! O+
  woensdag 13 november 2013 @ 21:40:51 #129
178193 Juicyhil
Bekende FOK!ker
pi_133233158
en wat de twee heren boven mij zeiden
Op dinsdag 9 augustus 2011 23:01 schreef SuperrrTuxxx het volgende:
Ik hou zoveel van jou, ik doe alles voor je! O+
pi_133236317
quote:
6s.gif Op woensdag 13 november 2013 19:42 schreef Chandler het volgende:
Wat vinden jullie van me scrippie? :D

http://pastebin.com/g2xTLLtP
Niet uitvoerbaar. Als ik de code copy-paste vanaf pastebin, krijg ik de melding "This archive is broken, get a good version". Dat blijkt dan over table structure, table data en file data te gaan.
  woensdag 13 november 2013 @ 22:41:15 #131
118585 Crutch
Filantroop || Taalzwengel
pi_133236396
quote:
6s.gif Op woensdag 13 november 2013 19:42 schreef Chandler het volgende:
Wat vinden jullie van me scrippie? :D

http://pastebin.com/g2xTLLtP
dafuq is this?
Je moeder is een hamster
  donderdag 14 november 2013 @ 07:44:33 #132
4159 GI
Nee ik heet geen JOE
pi_133243194
quote:
6s.gif Op woensdag 13 november 2013 19:42 schreef Chandler het volgende:
Wat vinden jullie van me scrippie? :D

http://pastebin.com/g2xTLLtP
What did I just read ?
pi_133243357
@Light: Klopt, dat de data onleesbaar is, moet zo zijn.
@Pakspul; bedankt voor je zeer nuttige reactie van 1 woord..
Juicyhil: Klopt, maar het script heeft ook niet als doel beschikbaar te zijn voor de 'hele' wereld... maar opbouwende tips kan ik altijd wat mee ;)
@Light: Het klopt inderdaad dat het om bestands en mysql data te gaan, en daaromheen zit een installer verwerkt, het rare is alleen dat het bestand bij mij deze melding niet geeft en bij jullie wel...
@Crutch: Versleutelde 'data' met een installer er aan vast geknoopt.
@GI: Read above :P

Het is vreemd dat de data op jullie systemen als 'broken' wordt weergegeven terwijl hier de code uitgevoerd kan worden, snap niet geheel waar het probleem zit maar daar moet ik dus nog mee spelen..
The people who lost my respect will never get a capital letter for their name again.
Like trump...
pi_133243477
Alvast een tip dan:
Magic constants horen te worden geschreven met hoofdletters, dus __DIR__ en __FILE__.
pi_133243601
Thanks, kleine vraag; jullie testen vast allemaal in een linux omgeving?

het probleem zit hem in PHP_EOL die in windows omgeving anders is dan in de linux omgeving....

[ Bericht 44% gewijzigd door Chandler op 14-11-2013 08:46:32 ]
The people who lost my respect will never get a capital letter for their name again.
Like trump...
  donderdag 14 november 2013 @ 08:48:49 #136
91039 mstx
2x1/2 = 1/2 x 1/2
pi_133243701
quote:
0s.gif Op donderdag 14 november 2013 08:39 schreef Chandler het volgende:
Thanks, kleine vraag; jullie testen vast allemaal in een linux omgeving?

het probleem zit hem in PHP_EOL die in windows omgeving anders is dan in de linux omgeving....
Op Debian werkt het niet.
Op donderdag 2 juli 2009 22:41 schreef RTB het volgende:
als ik elk rap"liedje" een kans moest geven was ik aan het eind van dit millennium nog bezig met het tempo waarin die kotshoop uitgebraakt wordt.
👾
pi_133243717
quote:
0s.gif Op donderdag 14 november 2013 08:48 schreef mstx het volgende:

[..]

Op Debian werkt het niet.
Daar ging ik al vanuit, alles anders dan Windows servers zal deze archive als broken betitelen... komt door PHP_EOL in PHP die onder windows anders is dan linux .....
The people who lost my respect will never get a capital letter for their name again.
Like trump...
pi_133244312
PHP_EOL gedraagt zich op ieder platform hetzelfde, namelijk: geef de line-ending van dit platform.

Als je specifiek een windows line-ending nodig hebt, slaat het dus nergens op om PHP_EOL te gebruiken.
pi_133244756
KomtTijd; Klopt! daar ben ik dan nu ook achter gekomen, dus ga ik daar mee aan de slag!
The people who lost my respect will never get a capital letter for their name again.
Like trump...
pi_133245474
quote:
6s.gif Op woensdag 13 november 2013 19:42 schreef Chandler het volgende:
jullie
Het doet pijn aan m'n ogen....
pi_133245586
quote:
4s.gif Op donderdag 14 november 2013 08:08 schreef Chandler het volgende:
@Light: Klopt, dat de data onleesbaar is, moet zo zijn.
@Pakspul; bedankt voor je zeer nuttige reactie van 1 woord..
Juicyhil: Klopt, maar het script heeft ook niet als doel beschikbaar te zijn voor de 'hele' wereld... maar opbouwende tips kan ik altijd wat mee ;)
@Light: Het klopt inderdaad dat het om bestands en mysql data te gaan, en daaromheen zit een installer verwerkt, het rare is alleen dat het bestand bij mij deze melding niet geeft en bij jullie wel...
@Crutch: Versleutelde 'data' met een installer er aan vast geknoopt.
@GI: Read above :P

Het is vreemd dat de data op jullie systemen als 'broken' wordt weergegeven terwijl hier de code uitgevoerd kan worden, snap niet geheel waar het probleem zit maar daar moet ik dus nog mee spelen..
Ok, reactie in één woord: MVC
pi_133256765
quote:
0s.gif Op donderdag 14 november 2013 10:40 schreef Pakspul het volgende:

[..]

Ok, reactie in één woord: MVC
Now you have two problems? Als in: hoe is MVC een oplossing voor de problemen van Chandler? :-)
Tegenwoordig moet je Dr. Ir. zijn om een beetje correct Nederlands te kunnen neerpleuren.
Abusing semicolons since 1987.
pi_133256931
quote:
0s.gif Op donderdag 14 november 2013 17:02 schreef rekenwonder het volgende:

[..]

Now you have two problems? Als in: hoe is MVC een oplossing voor de problemen van Chandler? :-)
Je mag toch op zijn minst wel je templates en je code gescheiden houden, niets problematisch aan hoor.
pi_133257454
quote:
14s.gif Op woensdag 13 november 2013 13:43 schreef boem-dikkie het volgende:

[..]

Heet de database ook hetzelfde?
Ik moest de cache legen in phpbb en daarna deed ie het..

Maar nu is mijn vraag waarom phpbb ./ voor de links zet voor smilies, ranks etc.

dus als mn smilie path is ingesteld als images/smilies (gewoon standaard dus)
dan maakt ie er ./images/smilies van waardoor de afbeelding niet word weergegeven.

??
  donderdag 14 november 2013 @ 18:08:35 #145
52200 ViPeRII
It's a good day to die
pi_133258486
quote:
0s.gif Op donderdag 14 november 2013 17:28 schreef Skunk-m het volgende:

[..]

Ik moest de cache legen in phpbb en daarna deed ie het..

Maar nu is mijn vraag waarom phpbb ./ voor de links zet voor smilies, ranks etc.

dus als mn smilie path is ingesteld als images/smilies (gewoon standaard dus)
dan maakt ie er ./images/smilies van waardoor de afbeelding niet word weergegeven.

??
Dat is opzich handig, aangezien je dan geen "harde" paden hoeft te gebruiken...
-- ViPeRII --
pi_133263210
quote:
0s.gif Op donderdag 14 november 2013 17:02 schreef rekenwonder het volgende:

[..]

Now you have two problems? Als in: hoe is MVC een oplossing voor de problemen van Chandler? :-)
Ja? slaat in mijn ogen compleet de plank mis :P want het enige wat het script moet doen is steeds het zelfde maar dan voor andere 'data' pakketten. Maar goed, MVC is gewoon een geil woord dat je overal voor moet gebruiken... voor countertjes, voor simpele contact formulieren, voor alles eigenlijk :+

quote:
14s.gif Op donderdag 14 november 2013 17:07 schreef KomtTijd... het volgende:
Je mag toch op zijn minst wel je templates en je code gescheiden houden, niets problematisch aan hoor.
Niet als het doel 1 bestand is, zonder includes! of wat voor externe bestanden dan ook ;) zelfs de afbeeldingen staan in CSS in base64 :@ ennuh op zich heb ik mijn html toch grotendeels gescheiden van php, alleen zit het wel verwerkt in het php bestand :+ als 1 lap tekst...

Maar goed, heb het probleem reeds 'denk ik' opgelost, nog even op een paar externe servers testen plus weer de nodige veranderingen (verbeteringen)...

Oh en kan natuurlijk altijd de nodige suggesties gebruiken, het kan altijd mooier, gemakkelijker of logischer :+

[ Bericht 4% gewijzigd door Chandler op 14-11-2013 20:25:41 ]
The people who lost my respect will never get a capital letter for their name again.
Like trump...
pi_133263368
MVC is zo 2012. MVP, MVVM, noem het maar op!
Tegenwoordig moet je Dr. Ir. zijn om een beetje correct Nederlands te kunnen neerpleuren.
Abusing semicolons since 1987.
pi_133312748
LOL wat is dan 2013? :P gewoon weer ouderwets? :P

Maaruh een update: http://pastebin.com/VtwXQHur voor diegene die het leuk vind om dingen te testen en opbouwend kritiek wil geven ;)

Oh pass is password123 :+
The people who lost my respect will never get a capital letter for their name again.
Like trump...
  zondag 17 november 2013 @ 00:58:18 #149
52200 ViPeRII
It's a good day to die
pi_133333681
quote:
15s.gif Op zaterdag 16 november 2013 11:39 schreef Chandler het volgende:
LOL wat is dan 2013? :P gewoon weer ouderwets? :P

Maaruh een update: http://pastebin.com/VtwXQHur voor diegene die het leuk vind om dingen te testen en opbouwend kritiek wil geven ;)

Oh pass is password123 :+
T werkt, op je quit knop na. deze brengt alles terug naar de 1e stap van je script.

Vervolgens zou het mooi zijn als je je script ook op andere "gebruikelijke" database systemen zou kunnen installeren, welke makkelijk te scripten zijn via PDO; Dus: mssql, postgres en sqlite, om de gebruiker wat meer opties te geven.

Daarnaast wil ik je ten zeerste aanraden het script netjes te maken, oftewel alles netjes in OOP schrijven in plaats van de procedural code die je nu geschreven hebt. Dat lijkt nu onzin, maar naar mate dat je script groeit, heb je daar altijd wat aan.

Ook dien je je unsets te testen om te kijken of ze wel dan niet geset zijn, je weet maar nooit wat er in je script gebeurd...Dat ben je bijvoorbeeld vergeten bij stap 7;

Ook heb ik nog de vraag waarom je de set_time_limit gebruikt..
Voor het mooie zou ik ook het aantal stappen wat proberen te verminderen.
-- ViPeRII --
pi_133340083
quote:
15s.gif Op zaterdag 16 november 2013 11:39 schreef Chandler het volgende:

Maaruh een update: http://pastebin.com/VtwXQHur voor diegene die het leuk vind om dingen te testen en opbouwend kritiek wil geven ;)
Je gebruikt twee keer "case 7:". Dat werkt niet, iedere case moet uniek zijn.

En $_SESSION unsetten is ook niet netjes. Da's een PHP superglobal, die moet je niet willen unsetten. Je kunt er wel een andere waarde aan geven, bijvoorbeeld null.
pi_133340467
Thanks voor jullie inhoudelijke reacties, de installer heeft een datablock en een script onderdeel, het script onderdeel ga ik niet oop maken, zou kunnen maar zie daar zelf het nut niet van in. Een keuze in database systemen ben ik zeker voor en gaat ook zeker verwerkt worden in het script en qua unset moet ik inderdaad nog wel het een en ander aanpassen... oh 2x case 7 had ik blijkbaar over het hoofd gezien... ooeps.. zelfde als set_time_limit, die heb ik gebruikt om te testen, iets wat bij kleine 'installers' echt niet nodig is maar bij installers van 30mb misschien wel, daar moet ik het allemaal nog op testen.

Paar vragen:
- Waarom stappen verminderen? zoveel stapjes zijn het niet? kijk eens naar programma's die je onder windows installeert? die hebben vaak ook vele stappen..
- Waarom OOP? zoveel code zit er niet in zeg maar... op zich zou't wel gemakkelijk zijn als ik er een ander ontwerp omheen wil gooien!
- Andere tips?

Oh de quit button is domweg bedoeld om de installer te resetten... je begint weer bij het begin want echt 'afsluiten' van het script zoals een programma gaat niet werken... tenzij ik window.close() ga gebruiken :+

thanks voor inhoudelijk feedback, maar hoe voelde de installer verder aan?
The people who lost my respect will never get a capital letter for their name again.
Like trump...
  zondag 17 november 2013 @ 13:36:56 #152
187069 slacker_nl
Sicko pur sang
pi_133340624
quote:
0s.gif Op zondag 17 november 2013 13:30 schreef Chandler het volgende:
Thanks voor jullie inhoudelijke reacties, de installer heeft een datablock en een script onderdeel, het script onderdeel ga ik niet oop maken, zou kunnen maar zie daar zelf het nut niet van in. Een keuze in database systemen ben ik zeker voor en gaat ook zeker verwerkt worden in het script en qua unset moet ik inderdaad nog wel het een en ander aanpassen... oh 2x case 7 had ik blijkbaar over het hoofd gezien... ooeps.. zelfde als set_time_limit, die heb ik gebruikt om te testen, iets wat bij kleine 'installers' echt niet nodig is maar bij installers van 30mb misschien wel, daar moet ik het allemaal nog op testen.

Paar vragen:
- Waarom stappen verminderen? zoveel stapjes zijn het niet? kijk eens naar programma's die je onder windows installeert? die hebben vaak ook vele stappen..
- Waarom OOP? zoveel code zit er niet in zeg maar... op zich zou't wel gemakkelijk zijn als ik er een ander ontwerp omheen wil gooien!
- Andere tips?

Oh de quit button is domweg bedoeld om de installer te resetten... je begint weer bij het begin want echt 'afsluiten' van het script zoals een programma gaat niet werken... tenzij ik window.close() ga gebruiken :+

thanks voor inhoudelijk feedback, maar hoe voelde de installer verder aan?
Je hoeft er m.i. niet per se OOP van te maken, maar het helpt wel (ik ben er wel een voorstander van). Maar je zou wel aparte functies kunnen maken, dat maakt je script makkelijker testbaar. Je kan dan elke functie testen op de werking. Dat lukt niet in je huidige model.

Verder zou ik wel iets aan je code conventies gaan doen:

1
2
3
4
5
6
7
8
9
$array = array ( 'ding' => .. , 'dat' => .. , 'zus' => .. , 'zo' => .. ); 
# zou ik herschrijven naar:

$array = array( 
    ding => .., 
    dat  => ..,
    zus  => ..,
    zo   => ..,
);

Dat leest en edit prettiger.
In theory there is no difference between theory and practice. In practice there is.
pi_133341618
quote:
0s.gif Op zondag 17 november 2013 13:36 schreef slacker_nl het volgende:
Je hoeft er m.i. niet per se OOP van te maken, maar het helpt wel (ik ben er wel een voorstander van). Maar je zou wel aparte functies kunnen maken, dat maakt je script makkelijker testbaar. Je kan dan elke functie testen op de werking. Dat lukt niet in je huidige model.
Mijn OOP is extreem brak, maak daarin vaak fouten en leer daar pas van als iemand er wat over zegt (mijn manier van leren). Ik zal eens kijken wat ik allemaal in functies kan stoppen wat er nu nog niet inzit...

quote:
Verder zou ik wel iets aan je code conventies gaan doen:

Dat leest en edit prettiger.
Klopt, daarin heb je totaal gelijk alleen hoeft een gemiddelde gebruiker van dit script niets met de data, zelfde als jij niets moet met de inhoudt van een installer anders dan de installer zijn werk laten doen... dus de data hoeft er niet mooi uit te zien (data block) maar qua scripten werkt het zeker beter...

verder nog grote php fouten gezien? nee toch? :D
The people who lost my respect will never get a capital letter for their name again.
Like trump...
  zondag 17 november 2013 @ 14:29:34 #154
187069 slacker_nl
Sicko pur sang
pi_133341873
quote:
0s.gif Op zondag 17 november 2013 14:19 schreef Chandler het volgende:

[..]

Mijn OOP is extreem brak, maak daarin vaak fouten en leer daar pas van als iemand er wat over zegt (mijn manier van leren). Ik zal eens kijken wat ik allemaal in functies kan stoppen wat er nu nog niet inzit...

[..]

Klopt, daarin heb je totaal gelijk alleen hoeft een gemiddelde gebruiker van dit script niets met de data, zelfde als jij niets moet met de inhoudt van een installer anders dan de installer zijn werk laten doen... dus de data hoeft er niet mooi uit te zien (data block) maar qua scripten werkt het zeker beter...

verder nog grote php fouten gezien? nee toch? :D
Het gaat me erom dat als je later in dat script kijkt dat het makkelijk edit en leest. Dat de gebruiker er niet in hoeft te lezen.. maar andere coders wel en die wil je wel helpen met schone, zelf-gedocumenteerde code.

Je kan toch heel makkelijk een class maken met functies en dan

1
2
3
4
$installer = new InstallerClass(
# named arguments
); 
$installer->run(); 
In theory there is no difference between theory and practice. In practice there is.
pi_133342363
Ook met OOP moet je je inlezen in code want ->run() zegt ook compleet niets, want wat gebeurd er in al die functies? dus denk dat documentatie van het script zelf stukken belangrijker is.. Al zou ik het wel leuk vinden om er een OOP versie van maken maken dat ik zo de schil (layout/vormgeving) on the fly kan aanpassen! *)
The people who lost my respect will never get a capital letter for their name again.
Like trump...
  zondag 17 november 2013 @ 15:02:56 #156
187069 slacker_nl
Sicko pur sang
pi_133342845
quote:
14s.gif Op zondag 17 november 2013 14:49 schreef Chandler het volgende:
Ook met OOP moet je je inlezen in code want ->run() zegt ook compleet niets, want wat gebeurd er in al die functies? dus denk dat documentatie van het script zelf stukken belangrijker is.. Al zou ik het wel leuk vinden om er een OOP versie van maken maken dat ik zo de schil (layout/vormgeving) on the fly kan aanpassen! *)
Je moet altijd je documentatie goed op orde hebben. Maar dat staat los van de manier van code. Al helpt het ook dat je code zichzelf uitlegt. Maar je functies moeten gewoon gedocumenteerd zijn.
In theory there is no difference between theory and practice. In practice there is.
pi_133345051
moeten is onzin, handig true; zal ik ook zeker gaan doen, maar blijkbaar vinden jullie het idee toch best leuk *) is een vervolg op een heel oud projectje (php2hex ;))
The people who lost my respect will never get a capital letter for their name again.
Like trump...
  zondag 17 november 2013 @ 16:21:37 #158
187069 slacker_nl
Sicko pur sang
pi_133345141
quote:
0s.gif Op zondag 17 november 2013 16:19 schreef Chandler het volgende:
moeten is onzin, handig true; zal ik ook zeker gaan doen, maar blijkbaar vinden jullie het idee toch best leuk *) is een vervolg op een heel oud projectje (php2hex ;))
Op m'n werk is het moeten. Want anders zegt de testsuite: je hebt de pod niet op orde en dan falen de tests.

Overigens zeg ik niks over je idee, want ik zou je script en je installer in aparte files frotten, maar dat is mijn mening.
In theory there is no difference between theory and practice. In practice there is.
pi_133348505
Waarom in aparte files? leg dat eens uit?
The people who lost my respect will never get a capital letter for their name again.
Like trump...
  zondag 17 november 2013 @ 18:43:06 #160
187069 slacker_nl
Sicko pur sang
pi_133349368
quote:
5s.gif Op zondag 17 november 2013 18:10 schreef Chandler het volgende:
Waarom in aparte files? leg dat eens uit?
Testbaarheid.
In theory there is no difference between theory and practice. In practice there is.
abonnement Unibet Coolblue Bitvavo
Forum Opties
Forumhop:
Hop naar:
(afkorting, bv 'KLB')