Skip to content

BUG: Add missing deprecated EncodedStreamObject functions#1140

Merged
MartinThoma merged 1 commit into1.xfrom
backport-bug-missing-deprecated-funcs
Jul 21, 2022
Merged

BUG: Add missing deprecated EncodedStreamObject functions#1140
MartinThoma merged 1 commit into1.xfrom
backport-bug-missing-deprecated-funcs

Conversation

@MasterOdin
Copy link
Copy Markdown
Member

@MasterOdin MasterOdin commented Jul 20, 2022

Backport of #1139 applied against 1.x. Please see that PR for any discussion / review.

Once merged and a new 1.x release, will fix the issue for #1138 preventing upgrading to latest 1.x release.

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 20, 2022

Codecov Report

Merging #1140 (fce7508) into 1.x (94b4d17) will decrease coverage by 0.05%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##              1.x    #1140      +/-   ##
==========================================
- Coverage   77.25%   77.19%   -0.06%     
==========================================
  Files          17       17              
  Lines        4393     4399       +6     
  Branches      820      820              
==========================================
+ Hits         3394     3396       +2     
- Misses        818      822       +4     
  Partials      181      181              
Impacted Files Coverage Δ
PyPDF2/generic.py 76.96% <33.33%> (-0.25%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 94b4d17...fce7508. Read the comment docs.

self.decoded_self = decoded
return decoded._data

def getData(self):
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The other deprecated functions in this file do not have the # pragma: no cover so I opted to not include it here like I did in #1139 (where all deprecated functions do have the pragma). Wasn't sure if should include it (and so that would probably get codecov to pass) or not include it.

@MartinThoma
Copy link
Copy Markdown
Member

oooh, very nice! Thank you for creating this branch - ok, then I'll make the update for 1.x as well :-)

@MartinThoma MartinThoma merged commit 2ceea0c into 1.x Jul 21, 2022
@MartinThoma MartinThoma deleted the backport-bug-missing-deprecated-funcs branch July 21, 2022 16:17
MartinThoma added a commit that referenced this pull request Jul 21, 2022
BUG: Add missing deprecated EncodedStreamObject functions (#1140)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants