Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bankruptcy courts might not use onclick attributes on links? #231

Closed
mlissner opened this issue Jan 19, 2018 · 1 comment
Closed

Bankruptcy courts might not use onclick attributes on links? #231

mlissner opened this issue Jan 19, 2018 · 1 comment

Comments

@mlissner
Copy link
Member

mlissner commented Jan 19, 2018

Got a bug report today on Twitter and the guy is having issues with case number 17-13797 in CAEB. Looking into it, it fails because the links in the docket don't have the onclick handler we look for:

  <a href="https://ecf.caeb.uscourts.gov/doc1/032026788825" 
    oncontextmenu="this.href=&quot;https://ecf.caeb.uscourts.gov/doc1/032026788825&quot;">281</a> 
  <br>
  <nobr>(2&nbsp;pgs)</nobr>
</td>

Could it be bankruptcy courts don't use these onclick handlers?

@johnhawkinson
Copy link
Collaborator

This is more clearly documented at https://twitter.com/ihatetonyy/status/954162045900894208 from @ihatetonyy and the stack trace (OCRd) is:

Uncaught TypeError : cannot read property ' match' of null pacer.js : 260
at Objec t. parseGoDLSFunction ( pace r js:260 )
at ContentDe l egate . f i ndAndstorePacerDoclds (conten_ delegate.js : 51)
at new contentoelega t e (content de)eeate.js:20 )
at content. j s : 15

because in https://github.com/freelawproject/recap-chrome/commits/f9852a2dacab02c50a8491586904156975be0c92 we assumed that link.getAttribute('onclick') would always return a String, but sometimes it returns null instead.

Anyhow, fixed in freelawproject/recap-chrome#40

Could it be bankruptcy courts don't use these onclick handlers?

It was never intended that the presence of onclick handlers be required. Merely parsed if they are present.

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

No branches or pull requests

2 participants