[Fixed bug]: Added null check on the parentElement in focusOtherModal()

[mario-nguyen]

If the modal is created without parent, the console will display error on closing function:

  modal.component.js:278 Uncaught TypeError: Cannot read property 'querySelectorAll' of null
    at ModalDirective.webpackJsonp.../../../../ngx-bootstrap/modal/modal.component.js.ModalDirective.focusOtherModal (modal.component.js:278)
    at modal.component.js:208
    at callbackRemove (modal.component.js:237)
    at ZoneDelegate.webpackJsonp.../../../../zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:424)
    at Object.onInvokeTask (core.es5.js:3881)
    at ZoneDelegate.webpackJsonp.../../../../zone.js/dist/zone.js.ZoneDelegate.invokeTask (zone.js:423)
    at Zone.webpackJsonp.../../../../zone.js/dist/zone.js.Zone.runTask (zone.js:191)
    at ZoneTask.invoke (zone.js:486)
    at timer (zone.js:1512)

[codecov]

Codecov Report

Merging #2612 into development will decrease coverage by 0.01%.
The diff coverage is 0%.

@@               Coverage Diff               @@
##           development    #2612      +/-   ##
===============================================
- Coverage        65.99%   65.98%   -0.02%     
===============================================
  Files              182      182              
  Lines             5011     5012       +1     
  Branches           790      791       +1     
===============================================
  Hits              3307     3307              
- Misses            1391     1392       +1     
  Partials           313      313

Impacted Files	Coverage Δ
src/modal/modal.component.ts	18.45% <0%> (-0.12%)	⬇️

---

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 101fb20...e9e8a68. Read the comment docs.

[Flood]

I'm experiencing the same error. Please merge.

[Flood]

@IlyaSurmay Can you release it to version 1.9.3? This issue was introduced 13 july and I have verified that this commit solves the issue.

[deggesim]

Same issue here. I'm using version 1.9.3

[valorkin]

@IlyaSurmay please resolve conflicts and push to new branch, so we can merge

[valorkin]

@mario-siama thanks again!
merged in #2792

[valorkin]

just so you know what we were doing lately :)

[valorkin]

https://ngx-bootstrap.surge.sh/

[Flood]

Thank you! Is it possible to merge into 1.9.4 so we don't need to use beta version?