Contributing code to WordPress core

Transcript

1. Contributing code to WP Core Peter Wilson peterwilson.cc @pwcc
3. Coding for core
4. Don’t hack WordPress core!
5. Don’t hack WordPress core! without contributing code back!
6. WPMelb Easter Egg <?php // inside get_post_class if ($title == "WordPress Melbourne") $classes[] = "wpmelb";
7. make.wordpress.org/core/handbook
8. What would happen without coding standards?
9. flic.kr/p/goGDXM (CC)
10. flic.kr/p/goGDXM (CC) We are WordPress’s passengers
11. WPMelb Easter Egg <?php // inside get_post_class $title = get_the_title($post_id); if ($title == "WordPress Melbourne") $classes[] = "wpmelb";
12. WPMelb Easter Egg <?php // inside get_post_class $title = get_the_title($post_id); if ("WordPress Melbourne" == $title) $classes[] = "wpmelb";
13. WPMelb Easter Egg <?php // inside get_post_class $title = get_the_title( $post_id ); if ( "WordPress Melbourne" == $title ) $classes[] = "wpmelb";
14. WPMelb Easter Egg <?php // inside get_post_class $title = get_the_title( $post_id ); if ( "WordPress Melbourne" == $title ) { $classes[] = "wpmelb"; }
15. WPMelb Easter Egg <?php // inside get_post_class $title = get_the_title( $post_id ); if ( "WordPress Melbourne" == $title ) { $classes[] = "wpmelb"; $classes[] = "wordpress-melbourne"; }
16. WPMelb Easter Egg <?php // inside get_post_class $title = get_the_title( $post_id ); if ( "WordPress Melbourne" == $title ) { $classes[] = "wpmelb"; }
17. Uploading a patch
19. #32831 WP Melb Easter Egg $classes[] = 'post-' . $post->ID; $title = get_the_title( $post_id ); if ( "WordPress Melbourne" == $title ) { $classes[] = "wpmelb"; } if ( ! is_admin() ) src/wp-includes/post-template.php
20. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff diff --git a/src/wp-includes/post-template.php b/src/wp-includes/post-template.php index 92ccf45..6d46d22 100644 --- a/src/wp-includes/post-template.php +++ b/src/wp-includes/post-template.php @@ -436,6 +436,11 @@ function get_post_class( $class = '', $post_id = null ) { } $classes[] = 'post-' . $post->ID; + $title = get_the_title( $post_id ); + if ( "WordPress Melbourne" == $title ) { + $classes[] = "wpmelb"; + } + if ( ! is_admin() ) $classes[] = $post->post_type; $classes[] = 'type-' . $post->post_type;
21. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff diff --git a/src/wp-includes/post-template.php b/src/wp-includes/post-template.php index 92ccf45..6d46d22 100644 --- a/src/wp-includes/post-template.php +++ b/src/wp-includes/post-template.php @@ -436,6 +436,11 @@ function get_post_class( $class = '', $post_id = null ) { } $classes[] = 'post-' . $post->ID; + $title = get_the_title( $post_id ); + if ( "WordPress Melbourne" == $title ) { + $classes[] = "wpmelb"; + } + if ( ! is_admin() ) $classes[] = $post->post_type; $classes[] = 'type-' . $post->post_type;
22. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff diff --git a/src/wp-includes/post-template.php b/src/wp-includes/post-template.php index 92ccf45..6d46d22 100644 --- a/src/wp-includes/post-template.php +++ b/src/wp-includes/post-template.php @@ -436,6 +436,11 @@ function get_post_class( $class = '', $post_id = null ) { } $classes[] = 'post-' . $post->ID; + $title = get_the_title( $post_id ); + if ( "WordPress Melbourne" == $title ) { + $classes[] = "wpmelb"; + } + if ( ! is_admin() ) $classes[] = $post->post_type; $classes[] = 'type-' . $post->post_type;
23. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff diff --git a/src/wp-includes/post-template.php b/src/wp-includes/post-template.php index 92ccf45..6d46d22 100644 --- a/src/wp-includes/post-template.php +++ b/src/wp-includes/post-template.php @@ -436,6 +436,11 @@ function get_post_class( $class = '', $post_id = null ) { } $classes[] = 'post-' . $post->ID; + $title = get_the_title( $post_id ); + if ( "WordPress Melbourne" == $title ) { + $classes[] = "wpmelb"; + } + if ( ! is_admin() ) $classes[] = $post->post_type; $classes[] = 'type-' . $post->post_type;
24. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff diff --git a/src/wp-includes/post-template.php b/src/wp-includes/post-template.php index 92ccf45..6d46d22 100644 --- a/src/wp-includes/post-template.php +++ b/src/wp-includes/post-template.php @@ -436,6 +436,11 @@ function get_post_class( $class = '', $post_id = null ) { } $classes[] = 'post-' . $post->ID; + $title = get_the_title( $post_id ); + if ( "WordPress Melbourne" == $title ) { + $classes[] = "wpmelb"; + } + if ( ! is_admin() ) $classes[] = $post->post_type; $classes[] = 'type-' . $post->post_type;
25. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff diff --git a/src/wp-includes/post-template.php b/src/wp-includes/post-template.php index 92ccf45..6d46d22 100644 --- a/src/wp-includes/post-template.php +++ b/src/wp-includes/post-template.php @@ -436,6 +436,11 @@ function get_post_class( $class = '', $post_id = null ) { } $classes[] = 'post-' . $post->ID; + $title = get_the_title( $post_id ); + if ( "WordPress Melbourne" == $title ) { + $classes[] = "wpmelb"; + } + if ( ! is_admin() ) $classes[] = $post->post_type; $classes[] = 'type-' . $post->post_type;
26. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff diff --git a/src/wp-includes/post-template.php b/src/wp-includes/post-template.php index 92ccf45..6d46d22 100644 --- a/src/wp-includes/post-template.php +++ b/src/wp-includes/post-template.php @@ -436,6 +436,11 @@ function get_post_class( $class = '', $post_id = null ) { } $classes[] = 'post-' . $post->ID; + $title = get_the_title( $post_id ); + if ( "WordPress Melbourne" == $title ) { + $classes[] = "wpmelb"; + } + if ( ! is_admin() ) $classes[] = $post->post_type; $classes[] = 'type-' . $post->post_type;
27. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff > ~/Downloads/32831.1.diff wptrac peterwilson$
28. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff > ~/Downloads/32831.1.diff wptrac peterwilson$
29. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff > ~/Downloads/32831.1.diff wptrac peterwilson$
30. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff > ~/Downloads/32831.1.diff wptrac peterwilson$
31. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff > ~/Downloads/32831.1.diff wptrac peterwilson$
32. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff > ~/Downloads/32831.1.diff wptrac peterwilson$
33. #32831 WP Melb Easter Egg wptrac peterwilson$ git diff > ~/Downloads/32831.1.diff wptrac peterwilson$
36. You retain the copyright. You license your code to WordPress
40. Let the code reviews begin
42. flic.kr/p/y35dEE (CC)
43. Hack WordPress core, contribute code back.
44. Hack WordPress docs, contribute docs back.
45. Translate WordPress, help spread the word.
46. Contribute however you can, however you feel comfortable.