Allow ordered_json in NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE macro

[ArashPartow]

Allow for ordered_json type to be used with the define macros provided in macro_scope.hpp.

Usage:

#include <cstdio>

#include <nlohmann/json.hpp>

struct mystruct
{
    int x = 1;
    double y = 2.0;
    std::string z = "three";
};

NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE
(
    mystruct,
    z,
    y,
    x
);

int main()
{
    mystruct ms;

    const auto msjson = nlohmann::to_string(nlohmann::ordered_json{{ "mystruct", ms }});

    const std::string expected = R"({"mystruct":{"z":"three","y":2.0,"x":1}})";

    printf("%s\n",msjson  .c_str());
    printf("%s\n",expected.c_str());

    return 0;
}

[github-actions]

🔴 Amalgamation check failed! 🔴

The source code has not been amalgamated. @ArashPartow
Please read and follow the Contribution Guidelines.

[coveralls]

coverage: 99.649%. remained the same
when pulling 4ac8d89 on ArashPartow:arashpartow/ordered_json_NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE
into a006a7a on nlohmann:develop.

[gregmarr]

This version in enum handling is what they should really be:

    template<typename BasicJsonType>                                                            \
    inline void from_json(const BasicJsonType& j, ENUM_TYPE& e)                                 \

[gregmarr]

See also #2843 and #2532

[nlohmann]

I agree with #4528 (comment) - a templated version would be a cleaner solution. Especially if more specializations are added such as #4517.

[ArashPartow]

@nlohmann Happy for you to close this PR, as #4517 might be the more appropriate PR to make the necessary changes.

[github-actions]

This pull request has been marked as stale because it has had no activity for 30 days. While we won’t close it automatically, we encourage you to update or comment if it is still relevant. Keeping pull requests active and up-to-date helps us review and merge changes more efficiently. Thank you for your contributions!

[nlohmann]

Closed in favor of #4597.