Skip to content

Commit a4313e3

Browse files
authored
Merge pull request #122 from Bowie375/main
fix: directly assign pdf_url to ArxivPaper._paper.pdf_url when it is None to enable correct pdf downloading
2 parents 78230d1 + 885b2ca commit a4313e3

File tree

1 file changed

+9
-4
lines changed

1 file changed

+9
-4
lines changed

paper.py

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,15 @@ def arxiv_id(self) -> str:
4040
def pdf_url(self) -> str:
4141
if self._paper.pdf_url is not None:
4242
return self._paper.pdf_url
43-
elif self._paper.links is not None:
44-
return self._paper.links[0].href.replace('abs','pdf')
45-
else:
46-
return f"https://arxiv.org/pdf/{self.arxiv_id}"
43+
44+
pdf_url = f"https://arxiv.org/pdf/{self.arxiv_id}.pdf"
45+
if self._paper.links is not None:
46+
pdf_url = self._paper.links[0].href.replace('abs','pdf')
47+
48+
## Assign pdf_url to self._paper.pdf_url for pdf downloading (Issue #119)
49+
self._paper.pdf_url = pdf_url
50+
51+
return pdf_url
4752

4853
@cached_property
4954
def code_url(self) -> Optional[str]:

0 commit comments

Comments
 (0)