Warnings about deprecated fields due to `JSON.stringify` usage about blazorgooglemaps

[valentasm1]

Very nice catch.
Maybe we could check google maps version and remove fields globaly on method?
I have just basic understanding of js, so not sure about right solution.

[AndreyTretyak]

I'm also don't know a lot about js. I think excluding fileds by default might be a bit risky since in the future Google Maps Api could add fields with the same names, or some customers might actually need those fields there.

I think a better approach would be to have an option of controling serialization behaivour that would allow avoiding such cases if needed.

For example define wrapper over JSON.stringify with override option and use it in the GoogleMapsComponents/wwwroot/js/objectManager.js:

const extendepbleStringify = function (obj, replacer, space) {
    if (window.blazorGoogleMapsBeforeStringify) {
          obj = window.blazorGoogleMapsBeforeStringify(obj);
    }
    return JSON.stringify(obj, replacer, space);
};

Then customers would be able to define their funtion to exclude fields that they have problem with, like this:

const removeDepricatedFieldsFromObject = function (obj) {
    if ('place_id' in obj) {
        delete obj.utc_offset
    }
    if ('opening_hours' in obj) {
        const hours = obj.opening_hours
        delete hours.open_now
    }
}

const removeDepricatedFields = function (obj) {
    if (Array.isArray(obj)) {
        for (const element of obj) {
            removeDepricatedFields(element)
        }
    } if (typeof obj === 'object' && obj !== null) {
        removeDepricatedFieldsFromObject(obj)
    }
}

window.blazorGoogleMapsBeforeStringify = function (obj) {
   removeDepricatedFields(obj);
   return obj;
};

This would also alow flexibility to adapt to future API changes and should not degrade performance too much for people who are not interested in excluding fields since warnings do not apear in most of the scenarios.

I'm not sure how common this approach are in js or is there are a nicer way, but if that's something you thin acceptable I would be happy to contribute this change. I'm also open for any other suggestions.

[AndreyTretyak]

My initial issue for google API was closed as "by design", so I've created another one with alternative suggestion https://issuetracker.google.com/issues/323230578

[valentasm1]

Sorry but i am on workation, so will take a look till 18 for sure. Especially it is not critical.

[AndreyTretyak]

No problem at all. Thank you for letting me know. I hope you'll have a great workation!

[valentasm1]

Reeleased
https://github.com/rungwiroon/BlazorGoogleMaps/releases/tag/4.1.0