Comments for digitalize.ca

Comment on Review: Tintin in America by Joen

Joen — Mon, 05 Dec 2011 08:31:09 +0000

Tintin comics high five, these are my favorites!

Though to be frank, the first two (or three depending on whether you count Tintin in Soviet) I find to be rather boring and dull. It’s not until Cigars of the Pharaoh (or whatever it’s translated to) that things get into gear.

Comment on On $wpdb->prepare by Mohammad Jangda

Mohammad Jangda — Thu, 24 Nov 2011 14:48:27 +0000

The problem is that prepare doesn’t do much other than sprintf the query with the values passed in. Developers can still do something like:

$query = $wpdb->prepare( "SELECT ID from {$wpdb->posts} WHERE ID = %d", $value );
error_log( 'Woo! I'm querying IDs! My query: ' . $query );
$id = $wpdb->get_col( $query );

Which is why this anti-pattern baffles me…

Comment on On $wpdb->prepare by Lloyd Budd

Lloyd Budd — Wed, 23 Nov 2011 23:19:53 +0000

You know how they come to that anti-pattern though, right? They want to isolate the query, so that if it was more easily cowboy coded — add and remove print and debug.

Wonder if prepare() having a debug/echo parameter would be useful, or too abstract… aside, does WP still not have as good of debug infrastructure as bbPress?

Comment on On $wpdb->prepare by Alex Mills (Viper007Bond)

Alex Mills (Viper007Bond) — Wed, 23 Nov 2011 19:32:30 +0000

Actually it’s a valid way to see if a post ID exists. :)

And +1 to your post.

Comment on On $wpdb->prepare by Mohammad Jangda

Mohammad Jangda — Wed, 23 Nov 2011 19:31:47 +0000

And, yes, the query in the example makes no sense.

Comment on jQTouch: Tap vs Click by Oskar

Oskar — Fri, 04 Nov 2011 12:53:29 +0000

You have an error in your code:
$(‘#mylink’).bind(clickEvent, function() {
e.preventDefault();
alert(‘Yay! You just ‘ + clickEvent + ‘ed me!’);
});

It should be:
function(e)

Comment on JavaScript Tip: Save me from console.log errors by Mohammad Jangda

Mohammad Jangda — Mon, 03 Oct 2011 04:19:30 +0000

Thanks for the tip. That’s definitely good to know!

Comment on JavaScript Tip: Save me from console.log errors by Jatinder Assi

Jatinder Assi — Tue, 27 Sep 2011 19:52:53 +0000

Actually the if stmt will always gets executed in IE 8, even when the developer tools window is open, thus resulting in the console object to be always a nul object.

Instead here is the solution that I used after modifying your code:
try { console.log('test'); } catch(err){ var console = {}; console.log = console.error = console.info = console.debug = console.warn = console.trace = console.dir = console.dirxml = console.group = console.groupEnd = console.time = console.timeEnd = console.assert = console.profile = function() {}; }

Comment on Co-Authors Plus by Laurence Elsdon

Laurence Elsdon — Sat, 24 Sep 2011 17:18:04 +0000

And now I’ve realised you can edit the co-author just by clicking on it…

Occam’s razor..

Feel free to delete this tree of comments

Comment on Co-Authors Plus by Laurence Elsdon

Laurence Elsdon — Sat, 24 Sep 2011 17:15:09 +0000

I did find a bit of a hack solution, I commented out line 112 of the plugin:

//add_action( ‘load-edit.php’, array( $this, ‘remove_quick_edit_authors_box’) );

This way I can use the quick edit to change the author that wordpress reads.

I set up a ghost account named ‘group’and manually set that as the default author for all co-authored posts. What would be best is if there was an option within the plugin to set a default author for any post with a co-author.