Skip to content
This repository has been archived by the owner on Mar 29, 2021. It is now read-only.

[Rollbar] TypeError: Cannot read property 'properties' of undefined #175

Closed
dsifford opened this issue Oct 2, 2016 · 1 comment · Fixed by #189 or #196
Closed

[Rollbar] TypeError: Cannot read property 'properties' of undefined #175

dsifford opened this issue Oct 2, 2016 · 1 comment · Fixed by #189 or #196
Labels

Comments

@dsifford
Copy link
Owner

dsifford commented Oct 2, 2016

This error has been thrown over 20 times on 4 sites beginning September 19 2016:

TypeError: Cannot read property 'properties' of undefined
File [redacted]/wp-content/plugins/academic-bloggers-toolkit/vendor/citeproc.js line 3 col 23341 in CSL.Engine.processCitationCluster
File [redacted]/wp-content/plugins/academic-bloggers-toolkit/lib/js/reference-list/index.js line 5 col 742 in e.value
File [redacted]/wp-content/plugins/academic-bloggers-toolkit/lib/js/reference-list/index.js line 4 col 10168 in t.n.insertInlineCitation
File [redacted]/wp-content/plugins/academic-bloggers-toolkit/lib/js/reference-list/index.js line 4 col 11301 in [anonymous]

any ideas here @fbennett ?

dsifford added a commit that referenced this issue Oct 15, 2016
- Add error and warning strings to i18n.
- Fix "cannot read property 'properties' of undefined error. (Closes
  #175).
- Add try/catch block to ReferenceList to improve error handling.
@dsifford dsifford mentioned this issue Oct 15, 2016
dsifford added a commit that referenced this issue Oct 15, 2016
**Changes:**

- Fix "double-sourcing" issue for citations inserted using DOI (Closes
  #187)
- Improve CSS naming conventions (using BEM methodology) + prune unused
  CSS (Closes #165)
- Remove automocks from tests + add ToggleSwitch componenet + tests
- Fix bug that threw exception when no results were found after
performing a pubmed search.
- Fix small CSS issues.
- Fix padding issue for selected items in reference list.
- Whitelist greenkeeper in travis.yml
- Add error and warning strings to i18n.
- Fix "cannot read property 'properties' of undefined error. (Closes #175).
- Add try/catch block to ReferenceList to improve error handling.
- Update wallaby configs + fix file extension issues in test files
- Update citeproc to latest version.
- Add `International Development Policy` style.
@dsifford
Copy link
Owner Author

Reopening due to this not being fully fixed in the latest release -- It's now fully fixed in the next release.

@dsifford dsifford reopened this Oct 16, 2016
dsifford added a commit that referenced this issue Oct 16, 2016
- [Fix] other half of "cannot read property 'properties' of undefined"
error. (Closes #175)
- [Fix] issue where exception is thrown when an invalid or unavailable
ISBN is queried. (Closes #194)
- [Fix] "this.registry.citationreg.citationById[r[0]] is undefined" error.
(Closes #197)
- [Improvement] Add support for resolving DataCite and mEDRA DOIs.
(Closes #193)
- Update tests.
- Refactor throughout.
dsifford added a commit that referenced this issue Oct 17, 2016
* Changes...

- Update broken webpack config.
- Add codecov to travis.yml

* Add dynamic injection of mobx-react-devtools. Closes #195

* Changes...

- [Fix] other half of "cannot read property 'properties' of undefined"
error. (Closes #175)
- [Fix] issue where exception is thrown when an invalid or unavailable
ISBN is queried. (Closes #194)
- [Fix] "this.registry.citationreg.citationById[r[0]] is undefined" error.
(Closes #197)
- [Improvement] Add support for resolving DataCite and mEDRA DOIs.
(Closes #193)
- Update tests.
- Refactor throughout.

* Changes...

- [Test] Improve test coverage in ReferenceList.
- [Chore] Fix wallaby configs
- [Fix] CSS class for fixed reference list to conform to BEM.
- [Fix] Remove Apple Safari recommendation from options page.
- [Chore] Improve ISSUE_TEMPLATE.md

Closes 198

* Changes...

- Add `Journal of Applied Clinical Medical Physics` style
- Bump to version 4.5.0 and build

Closes #198

* Update readme
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant