Modernize & fix #422

[StorytellerCZ]

Table of Contents generated with DocToc

• What
• Why

What

• Modernize code
• Remove unused dependency clone
• Fix Delete the name of collections in error messages #422
• Fix LGTM errors

Why

As far as I and my IDE has seen clone is no longer used, please correct me if I'm missing something here.

Then I have updated var to let and const and one loop for forEach loop.

Finally this PR also includes a proposed solution for #422.

[lgtm-com]

This pull request introduces 2 alerts and fixes 1 when merging 6b03016 into 55d3998 - view on LGTM.com

new alerts:

• 2 for Assignment to constant

fixed alerts:

• 1 for Unused variable, import, function or class

[harryadel]

You seem to be still making modifications. 😅 Please let me know when you're done.

[lgtm-com]

This pull request introduces 2 alerts and fixes 1 when merging 65f99cc into e8ec933 - view on LGTM.com

new alerts:

• 2 for Assignment to constant

fixed alerts:

• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 2 alerts when merging a164161 into e8ec933 - view on LGTM.com

fixed alerts:

• 1 for Comparison between inconvertible types
• 1 for Unused variable, import, function or class

[StorytellerCZ]

@harryadel sorry, noticed the LGTM errors so I went ahead and fixed them. There was actually one bug in that.

[lgtm-com]

This pull request fixes 2 alerts when merging 8ac80e7 into e8ec933 - view on LGTM.com

fixed alerts:

• 1 for Comparison between inconvertible types
• 1 for Unused variable, import, function or class

[lgtm-com]

This pull request fixes 2 alerts when merging afdb125 into e8ec933 - view on LGTM.com

fixed alerts:

• 1 for Comparison between inconvertible types
• 1 for Unused variable, import, function or class