PHP people... Quick question on why this no longer works... - GoFuckYourself.com

V_RocKs · 03-22-2009, 12:42 PM

This code used to work... Then my host upgraded my PHP and now it lo longer works... Pooh pooh!

Code:

if ($cmd == 'go' && ($url && $title && $blogs && $types))

The variables $url, $title, $blogs and $types all are arrays.

Again, this used to work yesterday.... Now it doesn't.

PHP was 5.2.5 and is now 5.2.9.

HorseShit · 03-22-2009, 12:44 PM

how is your blog network doing?

Deej · 03-22-2009, 12:46 PM

After their upgrade, did they get apache up and running properly?

A simple upgrade 'shouldnt' effect an already working script

V_RocKs · 03-22-2009, 12:54 PM

After upgrade everything else worked... except for this particular script. For some reason it doesn't return TRUE anymore. I believe it has something to do with the variables being arrays...

Blog network is going good.

Babaganoosh · 03-22-2009, 01:06 PM

Did they enable safe mode when they upgraded?

woj · 03-22-2009, 01:15 PM

they probably disabled register_globals

V_RocKs · 03-22-2009, 01:16 PM

Ah... was a globals problem.

gornyhuy · 03-22-2009, 02:01 PM

In case anyone is keyword searching, there actually is a backward-INcompatible change from PHP4.X to php5.X involving arrays.

In php5 If you don't explicitly declare a variable as an array and then later you try to use certain array functions on it (slice, splice, etc), it will not work.

I learned this the hard way when I upgraded a dedicated box with a bunch of legacy php code on it.

V_RocKs · 03-23-2009, 06:04 PM

Quote:

Originally Posted by gornyhuy

In case anyone is keyword searching, there actually is a backward-INcompatible change from PHP4.X to php5.X involving arrays.

In php5 If you don't explicitly declare a variable as an array and then later you try to use certain array functions on it (slice, splice, etc), it will not work.

I learned this the hard way when I upgraded a dedicated box with a bunch of legacy php code on it.

Thanks for the heads up!

Tempest · 03-23-2009, 06:17 PM

Quote:

Originally Posted by V_RocKs

This code used to work... Then my host upgraded my PHP and now it lo longer works... Pooh pooh!

Code:

if ($cmd == 'go' && ($url && $title && $blogs && $types))

The variables $url, $title, $blogs and $types all are arrays.

Again, this used to work yesterday.... Now it doesn't.

PHP was 5.2.5 and is now 5.2.9.

You should also write the code cleaner by using things like isset($url) instead of the way you're doing it.. Doing it the way you're doing it is going to be throwing a lot of errors you just don't see (unless you set the error reporting higher) but that could very well be being logged to the server error log. I learned that the hard way when I had a log filling up my disk space on a small virtual acount.

By the way.. If you ever want to source out your programming, I'd be interested in doing some of your stuff..

2012 · 03-23-2009, 06:22 PM

($url && $title && $blogs && $types)

that must be returning true

why not maybe change to

if($cmd == 'go' ){
if($url==true && $title==true && $blogs==true && $types==true){}
}
?

calmlikeabomb · 03-23-2009, 06:32 PM

This thread makes my balls hurts.

If they are arrays you should first check to make sure that's indeed what they are.

http://php.net/is_array

Also, try using a switch instead of a null else or long ass if-elseif-else chain.

http://php.net/switch

I'm happy the code you posted here fails

2012 · 03-23-2009, 06:35 PM

echo 'help me' . '<BR>' ;
echo 'help meeeeeeeeeeeeeeeeeeeeeeeeeeeeeee' ;

2012 · 03-23-2009, 06:37 PM

Quote:

Originally Posted by calmlikeabomb

I'm happy the code you posted here fails

how nice of you

Tempest · 03-23-2009, 06:48 PM

Quote:

Originally Posted by calmlikeabomb

This thread makes my balls hurts.

If they are arrays you should first check to make sure that's indeed what they are.

http://php.net/is_array

Also, try using a switch instead of a null else or long ass if-elseif-else chain.

http://php.net/switch

I'm happy the code you posted here fails

switch is for pussies... you're not a real programmer unless your code is unreadable.

2012 · 03-23-2009, 06:49 PM

Quote:

Originally Posted by Tempest

switch is for pussies... you're not a real programmer unless your code is unreadable.

no shit homie, don't you know they just re-instated the "goto" statement. Get with the times loser

Your point in mute

03-23-2009, 06:57 PM

if ($_GET['cmd'] == 'go' && (($_GET['url'] && $_GET['title']) && ($_GET['blogs'] && $_GET['types'])))

Win.

calmlikeabomb · 03-23-2009, 07:00 PM

^ That's a fail, because it'd require the array to be passed via the url.

Unless you're assigning an array to $_GET, which in case would also be a fail.

Plus you're still not checking to see if the variables are arrays. Wouldn't you want to do that before any processing occurs?

calmlikeabomb · 03-23-2009, 07:04 PM

I want to know what url, title, blogs, and types variable are part of so i can see the big picture. Wtf, does your script do.

2012 · 03-23-2009, 07:09 PM

Quote:

Originally Posted by Killswitch

if ($_GET['cmd'] == 'go' && (($_GET['url'] && $_GET['title']) && ($_GET['blogs'] && $_GET['types'])))

Win.

Quote:

Originally Posted by calmlikeabomb

^ That's a fail, because it'd require the array to be passed via the url.

Unless you're assigning an array to $_GET, which in case would also be a fail.

Plus you're still not checking to see if the variables are arrays. Wouldn't you want to do that before any processing occurs?

Quote:

Originally Posted by calmlikeabomb

I want to know what url, title, blogs, and types variable are part of so i can see the big picture. Wtf, does your script do.

http://www.gofuckyourself.com/showpo...00&postcount=9

Tempest · 03-23-2009, 07:42 PM

Quote:

Originally Posted by fartfly

no shit homie, don't you know they just re-instated the "goto" statement. Get with the times loser

Your point in mute

When I heard that I was exstatic... almost ended up with a cum stain...

Voodoo · 03-23-2009, 08:28 PM

This works. I'm not sure if it's proper for what you're doing or not, but maybe it helps.

Code:

$cmd=$_GET['cmd'];
$url=$_GET['url'];
$title=$_GET['title'];
$blogs=$_GET['blogs'];
$types=$_GET['types'];
$check=is_string($url)&&($title)&&($blogs)&&($types);

if(($cmd=='go')&&($check != FALSE)){
	echo "YAY!";
}else{
	echo "BOOO!";
}

Why · 03-23-2009, 08:45 PM

VooDoo, wtf you been up to lately man? good to see your still kickin.

NaughtyRob · 03-23-2009, 08:49 PM

Fucking geeks....

J/K I wish I knew php

uno · 03-23-2009, 11:37 PM

Quote:

Originally Posted by fartfly

no shit homie, don't you know they just re-instated the "goto" statement. Get with the times loser

Your point in mute

I'm on a grammar kick. Moot, not mute.

03-22-2009, 12:42 PM	#1
V_RocKs Damn Right I Kiss Ass! Industry Role: Join Date: Dec 2003 Location: Cowtown, USA Posts: 32,409	PHP people... Quick question on why this no longer works... This code used to work... Then my host upgraded my PHP and now it lo longer works... Pooh pooh! Code: if ($cmd == 'go' && ($url && $title && $blogs && $types)) The variables $url, $title, $blogs and $types all are arrays. Again, this used to work yesterday.... Now it doesn't. PHP was 5.2.5 and is now 5.2.9.

03-22-2009, 12:46 PM	#3
Deej I make pixels work Industry Role: Join Date: Jun 2005 Location: I live here... Posts: 24,386	After their upgrade, did they get apache up and running properly? A simple upgrade 'shouldnt' effect an already working script __________________ Deej's Designs n' What Not Hit me up for Design, CSS & Photo Retouching Icq#30096880

03-22-2009, 01:06 PM	#5
Babaganoosh ♥♥♥ Likes Hugs ♥♥♥ Industry Role: Join Date: Nov 2001 Location: /home Posts: 15,841	Did they enable safe mode when they upgraded? __________________ I like pie.

03-22-2009, 01:15 PM	#6
woj <&(©¿©)&> Industry Role: Join Date: Jul 2002 Location: Chicago Posts: 47,882	they probably disabled register_globals __________________ Custom Software Development, email: woj#at#wojfun#.#com to discuss details or skype: wojl2000 or gchat: wojfun or telegram: wojl2000 Affiliate program tools: Hosted Galleries Manager Banner Manager Video Manager Wordpress Affiliate Plugin Pic/Movie of the Day Fansign Generator Zip Manager

03-22-2009, 02:01 PM	#8
gornyhuy Chafed. Join Date: May 2002 Location: Face Down in Pussy Posts: 18,041	In case anyone is keyword searching, there actually is a backward-INcompatible change from PHP4.X to php5.X involving arrays. In php5 If you don't explicitly declare a variable as an array and then later you try to use certain array functions on it (slice, splice, etc), it will not work. I learned this the hard way when I upgraded a dedicated box with a bunch of legacy php code on it. __________________ icq:159548293

03-22-2009, 12:44 PM	#2
HorseShit Too lazy to set a custom title Join Date: Dec 2004 Posts: 17,513	how is your blog network doing?

03-22-2009, 12:54 PM	#4
V_RocKs Damn Right I Kiss Ass! Industry Role: Join Date: Dec 2003 Location: Cowtown, USA Posts: 32,409	After upgrade everything else worked... except for this particular script. For some reason it doesn't return TRUE anymore. I believe it has something to do with the variables being arrays... Blog network is going good.

03-22-2009, 01:16 PM	#7
V_RocKs Damn Right I Kiss Ass! Industry Role: Join Date: Dec 2003 Location: Cowtown, USA Posts: 32,409	Ah... was a globals problem.

03-23-2009, 06:22 PM	#11
2012 So Fucking What Industry Role: Join Date: Jul 2006 Posts: 17,189	coding the phps ($url && $title && $blogs && $types) that must be returning true why not maybe change to if($cmd == 'go' ){ if($url==true && $title==true && $blogs==true && $types==true){} } ? Last edited by 2012; 03-23-2009 at 06:26 PM.. Reason: i no know

03-23-2009, 06:32 PM	#12
calmlikeabomb Confirmed User Join Date: May 2004 Location: SW Palm Bay, Florida Posts: 1,323	This thread makes my balls hurts. If they are arrays you should first check to make sure that's indeed what they are. http://php.net/is_array Also, try using a switch instead of a null else or long ass if-elseif-else chain. http://php.net/switch I'm happy the code you posted here fails __________________ subarus.

03-23-2009, 06:35 PM	#13
2012 So Fucking What Industry Role: Join Date: Jul 2006 Posts: 17,189	echo 'help me' . '<BR>' ; echo 'help meeeeeeeeeeeeeeeeeeeeeeeeeeeeeee' ;

03-23-2009, 06:57 PM	#17
Killswitch - BANNED FOR LIFE Guest Posts: n/a	if ($_GET['cmd'] == 'go' && (($_GET['url'] && $_GET['title']) && ($_GET['blogs'] && $_GET['types']))) Win.

03-23-2009, 07:00 PM	#18
calmlikeabomb Confirmed User Join Date: May 2004 Location: SW Palm Bay, Florida Posts: 1,323	^ That's a fail, because it'd require the array to be passed via the url. Unless you're assigning an array to $_GET, which in case would also be a fail. Plus you're still not checking to see if the variables are arrays. Wouldn't you want to do that before any processing occurs? __________________ subarus. Last edited by calmlikeabomb; 03-23-2009 at 07:02 PM..

03-23-2009, 07:04 PM	#19
calmlikeabomb Confirmed User Join Date: May 2004 Location: SW Palm Bay, Florida Posts: 1,323	I want to know what url, title, blogs, and types variable are part of so i can see the big picture. Wtf, does your script do. __________________ subarus.

03-23-2009, 08:28 PM	#22
Voodoo ♥ ♦ ♣ ♠ Industry Role: Join Date: Sep 2002 Location: Porn Valley, CA Posts: 10,590	This works. I'm not sure if it's proper for what you're doing or not, but maybe it helps. Code: $cmd=$_GET['cmd']; $url=$_GET['url']; $title=$_GET['title']; $blogs=$_GET['blogs']; $types=$_GET['types']; $check=is_string($url)&&($title)&&($blogs)&&($types); if(($cmd=='go')&&($check != FALSE)){ echo "YAY!"; }else{ echo "BOOO!"; } __________________ "I'm selflessly supporting the common good, but only coincidentally looking out for No.1."

03-23-2009, 08:45 PM	#23
Why MFBA Industry Role: Join Date: Mar 2003 Location: PNW Posts: 7,230	VooDoo, wtf you been up to lately man? good to see your still kickin.

03-23-2009, 08:49 PM	#24
NaughtyRob Two fresh affiliate progs Industry Role: Join Date: Nov 2004 Location: Inside teen pussy Posts: 29,602	Fucking geeks.... J/K I wish I knew php __________________ [email protected] Skype: 17026955414 Vacares Web Hosting - Protect Your Ass with Included Daily Backups Last edited by NaughtyRob; 03-23-2009 at 08:51 PM..