Skip to content

Commit 9262e20

Browse files
committed
Handle thousand separator in public access
Fixes #414
1 parent 5603968 commit 9262e20

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

scholarly/author_parser.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -132,9 +132,9 @@ def _fill_public_access(self, soup, author):
132132
not_available = soup.find('div', class_='gsc_rsb_m_na')
133133
n_available, n_not_available = 0, 0
134134
if available:
135-
n_available = int(available.text.split(" ")[0])
135+
n_available = int(available.text.split(" ")[0].replace(",", ""))
136136
if not_available:
137-
n_not_available = int(not_available.text.split(" ")[0])
137+
n_not_available = int(not_available.text.split(" ")[0].replace(",", ""))
138138

139139
author["public_access"] = PublicAccess(available=n_available,
140140
not_available=n_not_available)

test_module.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -531,9 +531,9 @@ def test_public_access(self):
531531
100, thus requiring fetching information from a second page and 2) fill
532532
public access counts without fetching publications.
533533
"""
534-
author = scholarly.search_author_id("7x48vOkAAAAJ")
534+
author = scholarly.search_author_id("f4KlrXIAAAAJ")
535535
scholarly.fill(author, sections=['basics', 'public_access', 'publications'])
536-
self.assertGreaterEqual(author["public_access"]["available"], 110)
536+
self.assertGreaterEqual(author["public_access"]["available"], 1180)
537537
self.assertEqual(author["public_access"]["available"],
538538
sum(pub.get("public_access", None) is True for pub in author["publications"]))
539539
self.assertEqual(author["public_access"]["not_available"],

0 commit comments

Comments
 (0)