Page 2 of 2

Posted: Tue May 01, 2007 11:53 am
by Jenk
Oren wrote:Don't be shy... what's the name of this project? :P
I'm 99% positive it has been mentioned on this board before, and by that I mean that someone is involved with it. I'm not going to put my foot in it again :P

I've also just seen one of the contributors for the module I am reviewing has an @php.net address.. :?

Posted: Tue May 01, 2007 11:58 am
by Chris Corbyn
Jenk wrote:Aye, that's just about exactly what it is. Someone's framework that has just had more and more stuff glued on, but the people doing the glueing aren't too hot at development.
CodeIgniter?

Posted: Tue May 01, 2007 12:00 pm
by Oren
Jenk wrote:I've also just seen one of the contributors for the module I am reviewing has an @php.net address.. :?
OMG :? :? :?

Posted: Tue May 01, 2007 12:01 pm
by RobertGonzalez
Its gotta be osCommerce.

Posted: Tue May 01, 2007 12:02 pm
by Jenk
No, it's a framework based project, although osCommerce is a bit of a shocker too. Not CodeIgniter, never looked at their code.

Posted: Tue May 01, 2007 12:56 pm
by seodevhead

Code: Select all

sleep();
I think you know..

Posted: Tue May 01, 2007 1:08 pm
by Begby
Here are some of my favorite snippets from the app I work on that came from some of the other developers.


This is in the middle of a object and was put there as a joke I believe as calling it causes a fatal error (spray() doesn't exist)

Code: Select all

function check()
{
  if ($poop)
  {
     // stink
     $this->spray('freshener') ;
  }
}
This is the top of every page

Code: Select all

<SCRIPT language="JavaScript">
	function popup( url ){
	  cuteLittleWindow = window.open(url, "littleWindow", "location=0,scrollbars=1,width=450,height=300"); 
	}
</SCRIPT>
Derek quit about a year ago. I discovered this little gem a month ago and am very tempted to comment it out just to see what happens

Code: Select all

// DO NOT REMOVE THIS STATEMENT
// ALL HELL WILL BREAK LOOSE
// Derek 02/07/05
if( $reset )
{
	$this->get( $prodID ) ;
}
Here is another very informative comment

Code: Select all

/****************************************
* READ THIS!!!!                                           
*                                                          
* NEVER MODIFY THE INVENTORY OF A PRODUCT DIRECTLY          
* Call the inventoryChange() or inventoryUpdate() methods instead to track the changes
* 
* If you don't I swear on my mother's grave I am going to cut you with my knife, beat
* your wife with my tire iron, sell your children into slavery, then drive over to your house and kick
* your ugly useless dog
*                                                           
*******************************/
class tblProduct extends LawnGnome {

Posted: Tue May 01, 2007 1:13 pm
by RobertGonzalez
Dude, I didn't know lawnGnome's could be extended?!?! :?::!:

Posted: Tue May 01, 2007 2:01 pm
by Luke
I didn't either!! My lawn gnomes could use a few inches. :lol:

Posted: Wed May 02, 2007 3:30 am
by ptrpan
That guy is EEEEEEEEEEEEEMMMMMMMMMMMMMMMOOOOOOOOOOOO!!!!!!!!! :crazy:

Posted: Sat May 05, 2007 9:52 am
by GeertDD
It is not exactly a 'function', but I like type casting as a way of cleaning up data.

Code: Select all

$clean_id = (int) $_GET['dangerous_id'];