fix tests

This commit is contained in:
Matthias Pfefferle 2023-07-17 17:23:13 +02:00
parent d7e9d54063
commit 964ceee869
5 changed files with 53 additions and 54 deletions

View file

@ -1,5 +1,5 @@
# ActivityPub # # ActivityPub #
**Contributors:** [pfefferle](https://profiles.wordpress.org/pfefferle/), [mediaformat](https://profiles.wordpress.org/mediaformat/), [akirk](https://profiles.wordpress.org/akirk/), [automattic](https://profiles.wordpress.org/automattic/) **Contributors:** [automattic](https://profiles.wordpress.org/automattic/), [pfefferle](https://profiles.wordpress.org/pfefferle/), [mediaformat](https://profiles.wordpress.org/mediaformat/), [mattwiebe](https://profiles.wordpress.org/mattwiebe/), [akirk](https://profiles.wordpress.org/akirk/), [jeherve](https://profiles.wordpress.org/jeherve/), [nuriapena](https://profiles.wordpress.org/nuriapena/)
**Tags:** OStatus, fediverse, activitypub, activitystream **Tags:** OStatus, fediverse, activitypub, activitystream
**Requires at least:** 4.7 **Requires at least:** 4.7
**Tested up to:** 6.2 **Tested up to:** 6.2

View file

@ -30,13 +30,13 @@ class Shortcodes {
* @return string The post tags as hashtags. * @return string The post tags as hashtags.
*/ */
public static function hashtags( $atts, $content, $tag ) { public static function hashtags( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
$tags = \get_the_tags( $post->ID ); $tags = \get_the_tags( $item->ID );
if ( ! $tags ) { if ( ! $tags ) {
return ''; return '';
@ -65,13 +65,13 @@ class Shortcodes {
* @return string The post title. * @return string The post title.
*/ */
public static function title( $atts, $content, $tag ) { public static function title( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
return \wp_strip_all_tags( \get_the_title( $post->ID ), true ); return \wp_strip_all_tags( \get_the_title( $item->ID ), true );
} }
@ -85,9 +85,9 @@ class Shortcodes {
* @return string The post excerpt. * @return string The post excerpt.
*/ */
public static function excerpt( $atts, $content, $tag ) { public static function excerpt( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
@ -103,11 +103,11 @@ class Shortcodes {
$excerpt_length = ACTIVITYPUB_EXCERPT_LENGTH; $excerpt_length = ACTIVITYPUB_EXCERPT_LENGTH;
} }
$excerpt = \get_post_field( 'post_excerpt', $post ); $excerpt = \get_post_field( 'post_excerpt', $item );
if ( '' === $excerpt ) { if ( '' === $excerpt ) {
$content = \get_post_field( 'post_content', $post ); $content = \get_post_field( 'post_content', $item );
// An empty string will make wp_trim_excerpt do stuff we do not want. // An empty string will make wp_trim_excerpt do stuff we do not want.
if ( '' !== $content ) { if ( '' !== $content ) {
@ -188,22 +188,22 @@ class Shortcodes {
* @return string The post content. * @return string The post content.
*/ */
public static function content( $atts, $content, $tag ) { public static function content( $atts, $content, $tag ) {
// prevent inception $item = get_item();
remove_shortcode( 'ap_content' );
$post = get_item(); if ( ! $item ) {
if ( ! $post ) {
return ''; return '';
} }
// prevent inception
remove_shortcode( 'ap_content' );
$atts = shortcode_atts( $atts = shortcode_atts(
array( 'apply_filters' => 'yes' ), array( 'apply_filters' => 'yes' ),
$atts, $atts,
$tag $tag
); );
$content = \get_post_field( 'post_content', $post ); $content = \get_post_field( 'post_content', $item );
if ( 'yes' === $atts['apply_filters'] ) { if ( 'yes' === $atts['apply_filters'] ) {
$content = \apply_filters( 'the_content', $content ); $content = \apply_filters( 'the_content', $content );
@ -233,9 +233,9 @@ class Shortcodes {
* @return string The post permalink. * @return string The post permalink.
*/ */
public static function permalink( $atts, $content, $tag ) { public static function permalink( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
@ -248,12 +248,12 @@ class Shortcodes {
); );
if ( 'url' === $atts['type'] ) { if ( 'url' === $atts['type'] ) {
return \esc_url( \get_permalink( $post->ID ) ); return \esc_url( \get_permalink( $item->ID ) );
} }
return \sprintf( return \sprintf(
'<a href="%1$s">%1$s</a>', '<a href="%1$s">%1$s</a>',
\esc_url( \get_permalink( $post->ID ) ) \esc_url( \get_permalink( $item->ID ) )
); );
} }
@ -267,9 +267,9 @@ class Shortcodes {
* @return string The post shortlink. * @return string The post shortlink.
*/ */
public static function shortlink( $atts, $content, $tag ) { public static function shortlink( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
@ -282,12 +282,12 @@ class Shortcodes {
); );
if ( 'url' === $atts['type'] ) { if ( 'url' === $atts['type'] ) {
return \esc_url( \wp_get_shortlink( $post->ID ) ); return \esc_url( \wp_get_shortlink( $item->ID ) );
} }
return \sprintf( return \sprintf(
'<a href="%1$s">%1$s</a>', '<a href="%1$s">%1$s</a>',
\esc_url( \wp_get_shortlink( $post->ID ) ) \esc_url( \wp_get_shortlink( $item->ID ) )
); );
} }
@ -301,9 +301,9 @@ class Shortcodes {
* @return string * @return string
*/ */
public static function image( $atts, $content, $tag ) { public static function image( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
@ -325,7 +325,7 @@ class Shortcodes {
$size = $atts['type']; $size = $atts['type'];
} }
$image = \get_the_post_thumbnail_url( $post->ID, $size ); $image = \get_the_post_thumbnail_url( $item->ID, $size );
if ( ! $image ) { if ( ! $image ) {
return ''; return '';
@ -344,13 +344,13 @@ class Shortcodes {
* @return string The post categories as hashtags. * @return string The post categories as hashtags.
*/ */
public static function hashcats( $atts, $content, $tag ) { public static function hashcats( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
$categories = \get_the_category( $post->ID ); $categories = \get_the_category( $item->ID );
if ( ! $categories ) { if ( ! $categories ) {
return ''; return '';
@ -379,13 +379,13 @@ class Shortcodes {
* @return string The author name. * @return string The author name.
*/ */
public static function author( $atts, $content, $tag ) { public static function author( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
$name = \get_the_author_meta( 'display_name', $post->post_author ); $name = \get_the_author_meta( 'display_name', $item->post_author );
if ( ! $name ) { if ( ! $name ) {
return ''; return '';
@ -404,13 +404,13 @@ class Shortcodes {
* @return string The author URL. * @return string The author URL.
*/ */
public static function authorurl( $atts, $content, $tag ) { public static function authorurl( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
$url = \get_the_author_meta( 'user_url', $post->post_author ); $url = \get_the_author_meta( 'user_url', $item->post_author );
if ( ! $url ) { if ( ! $url ) {
return ''; return '';
@ -468,13 +468,13 @@ class Shortcodes {
* @return string The post date. * @return string The post date.
*/ */
public static function date( $atts, $content, $tag ) { public static function date( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
$datetime = \get_post_datetime( $post ); $datetime = \get_post_datetime( $item );
$dateformat = \get_option( 'date_format' ); $dateformat = \get_option( 'date_format' );
$timeformat = \get_option( 'time_format' ); $timeformat = \get_option( 'time_format' );
@ -497,13 +497,13 @@ class Shortcodes {
* @return string The post time. * @return string The post time.
*/ */
public static function time( $atts, $content, $tag ) { public static function time( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
$datetime = \get_post_datetime( $post ); $datetime = \get_post_datetime( $item );
$dateformat = \get_option( 'date_format' ); $dateformat = \get_option( 'date_format' );
$timeformat = \get_option( 'time_format' ); $timeformat = \get_option( 'time_format' );
@ -526,13 +526,13 @@ class Shortcodes {
* @return string The post date/time. * @return string The post date/time.
*/ */
public static function datetime( $atts, $content, $tag ) { public static function datetime( $atts, $content, $tag ) {
$post = get_item(); $item = get_item();
if ( ! $post ) { if ( ! $item ) {
return ''; return '';
} }
$datetime = \get_post_datetime( $post ); $datetime = \get_post_datetime( $item );
$dateformat = \get_option( 'date_format' ); $dateformat = \get_option( 'date_format' );
$timeformat = \get_option( 'time_format' ); $timeformat = \get_option( 'time_format' );

View file

@ -367,11 +367,7 @@ function get_item() {
return null; return null;
} }
if ( ! \in_array( $post->post_type, \get_post_types_by_support( 'activitypub' ), true ) ) { if ( 'publish' !== \get_post_status( $post ) ) {
return null;
}
if ( 'publish' !== $post->post_status ) {
return null; return null;
} }
@ -379,5 +375,9 @@ function get_item() {
return null; return null;
} }
if ( ! \in_array( \get_post_type( $post ), \get_post_types_by_support( 'activitypub' ), true ) ) {
return null;
}
return $post; return $post;
} }

View file

@ -1,5 +1,5 @@
=== ActivityPub === === ActivityPub ===
Contributors: pfefferle, mediaformat, akirk, automattic Contributors: automattic, pfefferle, mediaformat, mattwiebe, akirk, jeherve, nuriapena
Tags: OStatus, fediverse, activitypub, activitystream Tags: OStatus, fediverse, activitypub, activitystream
Requires at least: 4.7 Requires at least: 4.7
Tested up to: 6.2 Tested up to: 6.2

View file

@ -39,13 +39,12 @@ class Test_Activitypub_Shortcodes extends WP_UnitTestCase {
$post->post_date_gmt = current_time( 'mysql', 1 ); $post->post_date_gmt = current_time( 'mysql', 1 );
$post->post_title = 'Some title or other'; $post->post_title = 'Some title or other';
$post->post_content = '<script>test</script>hallo<script type="javascript">{"asdf": "qwerty"}</script><style></style>'; $post->post_content = '<script>test</script>hallo<script type="javascript">{"asdf": "qwerty"}</script><style></style>';
$post->post_status = 'publish';
$post->comment_status = 'closed'; $post->comment_status = 'closed';
$post->ping_status = 'closed'; $post->ping_status = 'closed';
$post->post_name = 'fake-page-' . rand( 1, 99999 ); // append random number to avoid clash $post->post_name = 'fake-page-' . rand( 1, 99999 ); // append random number to avoid clash
$post->post_type = 'page'; $post->post_type = 'page';
$post->post_password = 'abc';
$post->filter = 'raw'; // important! $post->filter = 'raw'; // important!
$post->post_password = 'abc';
$content = '[ap_content]'; $content = '[ap_content]';