search

mevdschee / php-crud-ui

Single file PHP script that adds a UI to a PHP-CRUD-API project
MIT License
123 stars 55 forks source link

the usage of compact/extract #36

Closed mevdschee closed 5 years ago

mevdschee commented 5 years ago

From (#35):

Note : the usage of compact/extract is (for me) difficult to read the code, and not IDE friendly.

I agree, and it should be avoided. The code should be rewritten in the same way I did with v2 for PHP-CRUD-API with PSR compliance, a build step and multiple files.

thipages commented 5 years ago

An intermediate step could be to add a value object representing the config

edited : php list may be also a possibility

thipages commented 5 years ago

If you could manage the intermediate step, it may facilitate a complementary javascript implementation I would like to start

mevdschee commented 5 years ago

a complementary javascript implementation I would like to start

Are you sure? Did you look at: https://github.com/nlware/vue-crud-ui?

thipages commented 5 years ago

yes I have looked at it. However I would like to build something different

  1. A specific minimalist javascript API for cruding a database with php-crud-api on server side (on the way)
  2. A view (gui) based on lighterhtml (a bit different than vuejs)
mevdschee commented 5 years ago

I would like to build something different

Sounds like a fun project. Enjoy!

mevdschee commented 5 years ago

I hope this helps :-)

thipages commented 5 years ago

Hi, yes it is fine, Thank you

Note : there is one unknown variable, the last I think ($record line 275)

mevdschee commented 5 years ago

Thank you fixed it!