Comments for David Walsh :: Legendary scribbles about JavaScript, HTML5, AJAX, PHP, CSS, and ∞.

Comment on Create a Zip File Using PHP by Chris

Chris — Thu, 02 Sep 2010 19:57:16 +0000

@daniel (or anyone else getting an “empty” zip file)

I too was getting zip files of the appropriate size, but appeared empty when viewed and and I tried to extract, windows was telling me the file was invalid. After looking at the ZipArchive manual on php.net, the problem stems from:

$zip->addFile($file,$file);

unless you have the function in the same directory as the files you want to zip, you will need to include the file path (which most people will if they use the function as an include). The 2nd parameter in addFile is the name of the file inside the zip, so if your $file var includes the path, that’s where the nested folder issue is coming from. So to solve this, I changed the code to :

$just_name = preg_replace("/(.*)\/?([^\/]+)/","$2",$file); $zip->addFile($file,$just_name);

which will strip out the file path (if any) and leave you only the file name for the 2nd variable in addFile. Now my zips open and extract just fine = )

Comment on Horizontally Center Your Website Structure Using CSS by Chris

Chris — Thu, 02 Sep 2010 19:13:19 +0000

Duuuuuuuuuuuuuude!

Big thanks to you!!! I was ready to punch my keyboard in the left ventricle until I found this.

Comment on Animated AJAX Record Deletion Using jQuery by Danny

Danny — Thu, 02 Sep 2010 14:11:35 +0000

Also, after reading a few other posts,

For those who want the ‘Are you sure message’ to appear on click, you can actually pulll in a standard javascript call:

onclick=”return confirm(\’Are your sure you want to Delete this Item?\’)”>

Add it to the href and hey presto.

Comment on MooTools Plugin: Event.Mock by Elad Ossadon

Elad Ossadon — Thu, 02 Sep 2010 13:27:12 +0000

Very nice!
I would definitely use it for testing, but not in production code. I think that firing events manually isn’t a best practice. I’d rather dividing into two functions – a listener and a performer, where the listener calls the performer (with or without event data). Then, I can call the performer from wherever I need in the code.

Comment on Quick Tip: Object Indexing vs. Array Collection by Richard K. Szabo

Richard K. Szabo — Thu, 02 Sep 2010 08:44:48 +0000

As pointed out,
if($found_keywords[$keyword]) { //or array_key_exists($keyword,
requires isset() to test.

Good ideas about checking up the items in JS!

Comment on 7 Quick CSS Enhancements for Better User Experience by Beben

Beben — Thu, 02 Sep 2010 04:19:29 +0000

share knowledge in the beauty of argument ;))
all is good if in accordance with the purposes and needs :)
bye bye :-h

Comment on 7 Quick CSS Enhancements for Better User Experience by Jeremy Carlson

Jeremy Carlson — Thu, 02 Sep 2010 01:42:52 +0000

I thought the new thing was to style :focus the same way you would style :hover, therefore eliminating the border wouldn’t matter. I’m not familiar with what vision-impaired people go through, so is there something that the border does for them, that the same style from :hover wouldn’t cover?

Comment on Duplicate the jQuery Homepage Tooltips by cancel bubble

cancel bubble — Wed, 01 Sep 2010 22:17:10 +0000

There’s no background on the tooltips in PC FF 3.6.8

Comment on Create a Zip File Using PHP by Mike

Mike — Wed, 01 Sep 2010 21:05:56 +0000

I’m getting this same behavior. Work on Ubuntu archive mounter, but not in archive manager. Doesn’t work on windows.

Here’s the error message on linux:

Error: /home/michael/Downloads/dsat-assessment.zip: Can not open file as archive

Errors: 1

Any suggestions?

Comment on An Open Letter to You, Webmaster by John

John — Wed, 01 Sep 2010 18:55:58 +0000

Laughable, but also true.