sachin-handiekar / jInstagram / 319

62%

Pull #182

travis-ci

deprecate TagMediaFeed and the API using it

Motivation

TagMediaFeed encapsulates exactly the same state as MediaFeed. Having
two separate classes makes it difficult to:
- reason about
- process a collection of both MediaFeed and TagMediaFeed

Point #2 is exacerbated by the returned MediaFeed type from the
InstagramClient#getRecentMediaNextPage API. This API flow starts with
an initial TagMediaFeed and fetches subsequence MediaFeed objects
which makes using Java 8 streams difficult.

Solution

Deprecate the use of TagMediaFeed in favor of MediaFeed. APIs were
added to InstagramClient to fetch tagged media with a MediaFeed
response type.

Deprecation notices were added conveying API removal in next major
version (likely 2) and what APIs you should use now to avoid
deprecation warnings.

fixes #179

21 of 21 new or added lines in 1 file covered. (100.0%)

945 of 1660 relevant lines covered (56.93%)

5.09 hits per line