Skip to content
This repository was archived by the owner on Aug 16, 2024. It is now read-only.

Add Post/RelatedUrls#36

Merged
tangrufus merged 8 commits intomasterfrom
related-urls
Apr 24, 2017
Merged

Add Post/RelatedUrls#36
tangrufus merged 8 commits intomasterfrom
related-urls

Conversation

@tangrufus
Copy link
Copy Markdown
Member

No description provided.

Comment thread src/Post/RelatedUrls.php Outdated
return 0;
}

return url_to_postid($url);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

url_to_postid() is prohibited, please use wpcom_vip_url_to_postid() instead.

Comment thread src/Post/RelatedUrls.php

$related = [];
foreach ($terms as $term) {
$link = get_term_link($term, $taxonomy);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

get_term_link() is prohibited, please use wpcom_vip_get_term_link() instead.

Comment thread src/Post/Finder.php
return null;
}

$postId = url_to_postid($cleanUrl);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

url_to_postid() is prohibited, please use wpcom_vip_url_to_postid() instead.

@tangrufus tangrufus force-pushed the related-urls branch 2 times, most recently from eaae14f to 79a591c Compare April 24, 2017 16:30
Comment thread src/Post/Finder.php
return;
}

$postId = url_to_postid($cleanUrl);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

url_to_postid() is prohibited, please use wpcom_vip_url_to_postid() instead.

@tangrufus tangrufus merged commit f5c6fd8 into master Apr 24, 2017
@tangrufus tangrufus deleted the related-urls branch April 29, 2017 11:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants