PlaceRequest.Builder(PlaceRequest request) makes a "shared copy" instead of a "isolated copy"

[thomasreisenbichler]

When instantiating a new PlaceRequest with "new PlaceRequest.Builder(request)" the parameter map of the base request is directly assigned to the new PlaceRequest which results in a strange situation. If you add a parameter to the second request, then the parameter change is reflected on the first request too.

So the following statements result in a green test, which cannot be correct.

    PlaceRequest request = new PlaceRequest.Builder().nameToken("nameToken").with("paramkey", "paramvalue").build();
    PlaceRequest copy = new PlaceRequest.Builder(request).with("copyparamkey", "copyparamvalue").build();

    assertEquals(request, copy);
    assertEquals(request.getParameter("copyparamkey", null), "copyparamvalue");

[christiangoudreau]

Closing, this has been fixed. Reopen if not.