aj-techsoul / ELECTRON-4-PHP

An easiest way to use php in electron
MIT License
203 stars 46 forks source link

Issue with this existing #10

Closed lordraz0r closed 5 years ago

lordraz0r commented 5 years ago

So what possible use case on this earth would require this and do you realize the performance bottleneck doing this would cause? Electron is a wrapper for javascript applications to run as desktop applications... NOT A THING YOU SHOULD BUTCHER PHP INTO!!!

aj-techsoul commented 5 years ago

See this was made to make life easier for php developers. People can't change their whole php code into JS just to for electron. This is just there as an Option. If you don't Like then please don't use it. For you there is Electron. This for people who were searching for an option and i provided them. This is just for Solution for now, for performance, we will see later. As per now, Performance is also not an issue as it works with PHP 7.2 Thanks.

lordraz0r commented 5 years ago

No this is not closed! "Performance is also not an issue as it works with PHP 7.2" is completely false! You are still basically running an entire instance of a PHP server for zero reason. Using Electron in this way is like using a screwdriver to smash a nail into the wall. Also I know of no competent PHP developers that can't code Javascript.

aj-techsoul commented 5 years ago

7.2 works in this and i have showed the proof also. But if you think performance is an issue. Then please tell me in what cases the performance is getting lowered. And I am a PHP, NODEJS,Python Programmer so i will try to fix this, otherwise i will make something like this again in Python instead of using NodeJS. Because PHP,Nodejs and python is a living language, and i will always support it for being more powerful and vast.

Please share me the facts not philosophical thoughts.

btw please watch this about NodeJS and JavaScript from an Expert. https://www.youtube.com/watch?v=VBfvnKDlZMw

userabuser commented 5 years ago

@lordraz0r you have an issue with this existing? That's rich. Well done for being supportive and respectful to open source contributors.

Just because you cannot fathom a use-case (due to lack of imagination among other things), does not mean others cannot.

I have a specific, experimental, use-case where embedding PHP into an Electron application to run a particular variation of CMS software for a particular educational purpose is required whereby:

A) we cannot assume the user has PHP installed B) we cannot assume the user is willing to install PHP locally C) using some other means, i.e. Docker, Vagrant, etc, will not suffice

There are other ways this can be achieved, some more efficient than others, however not all methodologies apply under certain conditions which is why alternative options exist, like this one, which may meet the needs of a particular use-case.

No this is not closed!

Who died and made you king?


@AJ-TechSoul don't let comments like this dissuade you from developing this or anything else.

juppwerner commented 5 years ago

AJ-TechSoul and userabuser, I totally agree with both of you.

Thanks a lot for this interesting repository!

Regards Joachim

shahidmau commented 4 years ago

I congratulate and thanks for this awesome library. There are several reasons to have this.

Thanks again Ajit Kumar keep working on this.