search

airbnb / DeepLinkDispatch

A simple, annotation-based library for making deep link handling better on Android
http://nerds.airbnb.com/deeplinkdispatch/
4.37k stars 409 forks source link

Fix NPE in issue #351 #352

Closed EricKuck closed 2 years ago

EricKuck commented 2 years ago

Issue #351 reports an NPE when query parameters do not have values, such as schema://mycompany/search?q. This PR fixes that issue by using empty strings for this case.

rossbacher commented 2 years ago

@EricKuck Really appreciate the fix did not get to that issue yet but could you also add a unit test so that issue does not slip through the cracks again?

EricKuck commented 2 years ago

Sure, test added

rossbacher commented 2 years ago

Thank you @EricKuck!