Opened 9 years ago

Closed 7 years ago

Last modified 7 years ago

#1079 closed defect (bug) (fixed)

Exclude Tweets from Insert/Edit Link search

Reported by: siobhan's profile siobhan Owned by: iandunn's profile iandunn
Milestone: Priority: normal
Component: WordCamp Site & Plugins Keywords: good-first-bug has-patch

Description (last modified by siobhan)

The search feature on Insert/Edit link is rendered useless because the search includes tweets. It would be useful to exclude these from the search (see image for what I mean).

Attachments (2)

1079.diff (1.8 KB) - added by JasonM4563 9 years ago.
1079.2.diff (1.9 KB) - added by JasonM4563 8 years ago.
Updated Patch

Download all attachments as: .zip

Change History (10)

#1 @siobhan
9 years ago

  • Description modified (diff)

#2 @iandunn
9 years ago

  • Keywords needs-patch good-first-bug added
  • Owner set to iandunn
  • Status changed from new to accepted


#3 @iandunn
9 years ago

The tweets (and related) post types are part of Tagregator, for anyone looking to submit a patch.

9 years ago

#4 @JasonM4563
9 years ago

  • Keywords has-patch added; needs-patch removed

Added a filter to remove the CPTs from the search results when adding a link to a post.

#5 @iandunn
9 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

Great, thanks AJ :)

Commited in plugins 1275689

#6 @JasonM4563
8 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Found a better way to resolve this which is more future proof and better utilizes the existing class variable. Reopening to submit new patch.

8 years ago

Updated Patch

#7 @coreymckrill
7 years ago

  • Resolution set to maybelater
  • Status changed from reopened to closed

The first patch seems to be working. The second one could be incorporated into a larger set of improvements to the plugin later, but it's not a priority now.

#8 @SergeyBiryukov
7 years ago

  • Resolution changed from maybelater to fixed

The only difference between 1079.2.diff and appears to be the placement of exclude_from_insert_link_results() in tagregator.php vs. tggr-media-source.php, which doesn't seem like much of an improvement, so I think it's safe to close this as fixed in comment:5.

Note: See TracTickets for help on using tickets.