fixes overwriting of namespaces w/ null value per mock

jakerella / jquery-mockjax

The jQuery Mockjax Plugin provides a simple and extremely flexible interface for mocking or simulating ajax requests and responses

Other

2.12k stars 376 forks source link

fixes overwriting of namespaces w/ null value per mock #315

Closed tomeara closed 7 years ago

tomeara commented 7 years ago

This adds a condition to evaluate if a mock's namespace has been explicitly set to null. If that is the case, the url does not get the mockjaxSettings.namespace added to it. See issue #314 for more detail.

This PR is closely aligned with #313 but should be able to be merged before or after.

jakerella commented 7 years ago

Thank you again for the contribution!! I ended up doing this in the line above instead and checking for undefined explicitly on that setting, I hope you don't mind. That said, I really appreciate the bug reports and fixes!

Closed in favor of #320.