missing php extensions in composer.json

[tenzap]

I noticed that the framework uses ctype_*, mb_*, hash_*, json_*, session_* which are part of the ctype, mbstring, json, session & hash extensions.

These extensions are not in the composer.json require section.

    "require": {
....
        "ext-ctype": "*",
        "ext-hash": "*",
        "ext-json":  "*",
        "ext-mbstring": "*",
        "ext-session":  "*",

Maybe it would be good to add them?

[gxgpet]

Well, yes and no.

One could use the framework without the need for any of the above extensions, as they are used in certain files. For those, the composer installation could fail if they lack at least one of the mentioned PHP extensions in require.

[tenzap]

Maybe then put it in "suggest"? I saw mbstring and iconv are not mandatory. The others depend on whichever lib of ci is used?

[otengkwame]

I saw mbstring and iconv are not mandatory. The others depend on whichever lib of ci is used?

I think currently is not really mandatory to require those in composer. CI is very flexible on this