Deprecate the JSON parser

contao / manager-plugin

Contao Manager Plugin

GNU Lesser General Public License v3.0

4 stars 9 forks source link

Deprecate the JSON parser #6

Closed leofeyer closed 7 years ago

leofeyer commented 7 years ago

IMO we should encourage people to primarily use the ContaoManager\Plugin class to register bundles, routes and configuration, therefore we should deprecate the JSON parser and the bundles.json file. Most people don't use a bundles.json file anyway.

@contao/developers What do you think?

Toflar commented 7 years ago

And they shouldn't because I'm not sure if opcode can optimize this. If you are returning a regular key value array I'm pretty sure opcode can improve speed there so I'm all in for that deprecation.

aschempp commented 7 years ago

fine with me. We just need the ini parser for legacy extensions.

leofeyer commented 7 years ago

Deprecated in b8f567760289f10701f334f8f907217f8452ed8a.