Print deprecation message only on usage of the returned .js() and .css() value

[trygve-lie]

From V3 to V4 we deprecated the return value from the .js() and .css() methods. In V5 we will remove that these return a value.

Due to this an deprecation message was added but the deprecation message is printed when these methods are used. Calling these methods to set a value will trigger the deprecation message which are not quite what we want.

This little hack makes it so that the deprecation message is printed only when one try to use the returned value from these methods.

This will not trigger a deprecation message:

const css = podlet.css({ value: '/public/css/main.css' });

This will trigger a deprecation message:

const css = podlet.css({ value: '/public/css/main.css' });
app.get(css, (req, res) => {
    [ .... ]
});

[alexanbj]

Nice work 👍

[digitalsadhu]

It's filthy, but I like it ;)

[stipsan]

Hehe 🙈