Skip to content

ROB: Guard against None-value in _get_outlines#1060

Merged
MartinThoma merged 3 commits intomainfrom
issue-1059
Jul 9, 2022
Merged

ROB: Guard against None-value in _get_outlines#1060
MartinThoma merged 3 commits intomainfrom
issue-1059

Conversation

@MartinThoma
Copy link
Copy Markdown
Member

See #1059

@MartinThoma
Copy link
Copy Markdown
Member Author

So far, this PR only prevents a crash. It still does not get the outlines, although Atril manages to get them:

image

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 5, 2022

Codecov Report

Merging #1060 (ba043a4) into main (1e9c4dd) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1060   +/-   ##
=======================================
  Coverage   90.89%   90.89%           
=======================================
  Files          24       24           
  Lines        4515     4515           
  Branches      923      923           
=======================================
  Hits         4104     4104           
  Misses        268      268           
  Partials      143      143           
Impacted Files Coverage Δ
PyPDF2/_reader.py 91.40% <100.00%> (ø)

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 1e9c4dd...ba043a4. Read the comment docs.

@MartinThoma MartinThoma added the is-robustness-issue From a users perspective, this is about robustness label Jul 5, 2022
@MartinThoma MartinThoma merged commit 439c749 into main Jul 9, 2022
@MartinThoma MartinThoma deleted the issue-1059 branch July 9, 2022 11:49
MartinThoma added a commit that referenced this pull request Jul 10, 2022
New Features (ENH):
-  Add PageObject._get_fonts (#1083)
-  Add support for indexed color spaces / BitsPerComponent for decoding PNGs (#1067)

Performance Improvements (PI):
-  Use iterative DFS in PdfWriter._sweep_indirect_references (#1072)

Bug Fixes (BUG):
-  Let Page.scale also scale the crop-/trim-/bleed-/artbox (#1066)
-  Column default for CCITTFaxDecode (#1079)

Robustness (ROB):
-  Guard against None-value in _get_outlines (#1060)

Documentation (DOC):
-  Stamps and watermarks (#1082)
-  OCR vs PDF text extraction (#1081)
-  Python Version support
-  Formatting of CHANGELOG

Developer Experience (DEV):
-  Cache downloaded files (#1070)
-  Speed-up for CI (#1069)

Maintenance (MAINT):
-  Set page.rotate(angle: int) (#1092)
-  Issue #416 was fixed by #1015 (#1078)

Testing (TST):
-  Image extraction (#1080)
-  Image extraction (#1077)

Code Style (STY):
-  Apply black
-  Typo in Changelog

Full Changelog: 2.4.2...2.4.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is-robustness-issue From a users perspective, this is about robustness

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant