Properly set parcel_pickup to yes in deprecation fix

[riQQ]

Fixes #364

[tyrasd]

Hey. Thanks for this suggestion. I solved it slightly differently in 609c017. Basically, I only removed the upgrade path of the parcel_mail_in case, which was causing the issue. Your solution would have also worked and would arguably have been an even better solution, because it would have preserved the upgrade path for mail-in-only parcel lockers features. I did however go with the other approach, because adding parcel_pickup=yes is somewhat discouraged by the osm wiki, see also #339 (comment). Since drop-off only parcel lockers are relatively rare, I think this solution is acceptable for the moment.

Ideally, there would be a way to indicate in the deprecated.json that upgrading a tag should explicitly not set a particular tag (e.g. something like this: "replace": {"amenity": "parcel_locker", "parcel_mail_in": "yes", "parcel_pickup": null}), but that would need to be implemented as a separate enhancement of this repository.

//edit: I see that there also exists the possibility to specify a replacement in a tagging presets. This should allow to properly implement a dedicated upgrade path for all 3 kinds of parcel lockers.
//edit2: Sadly, the solution mentioned in my first edit doesn't work here, because there doesn't exist a specific preset for the dropoff-only parcel lockers to replace the old tagging variant.