Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automatic SQL execution, including table creation and index / unique keys creation / deletion #214

Open
ShadeSlider opened this issue Jul 22, 2014 · 16 comments

Comments

@ShadeSlider
Copy link

Всем привет!

Наконец-то, спустя 6 лет использования onPHP, у меня дошли руки добавить в него то, чего а мне не хватало с самого начала.

А именно:

  1. Автоматическое выполнение SQL запросов в базе
  2. Создание таблиц и индексов / уникальных ключей
  3. Запись выполняемого SQL в файл

Собственнно, вот мой форк: https://github.com/ShadeSlider/onphp-framework
В Readme.md форка подробно описано, что добавлено, и как с этим работать.

Если контрибьютеры посчитают что-то из этого достойным для влития в основной фреймворк - готов потратить время на оформление pull request.

Код местами, возможно, нужно будет порефакторить, но общая картина должна быть понятна.

В форке есть другие мелкие изменения / добавления, не связанные с генерацией кода. Все, что связанно с генерацией, не должно ломать BC.

Что скажете?

@glushchenko
Copy link

Господа, почему тишина? Дельный форк, хорошо бы влить в основную ветку.

@dovg
Copy link
Member

dovg commented Aug 4, 2014

Хороший форк, только похоже влить некому.

Друзья, давайте проведем перепись выживших :)
@ssserj, @suquant, @AlexeyDsov - вы живы? Все еще используете onphp?

ps. Я на php в последнее время не пишу и у меня, к сожалению, нет времени на поддержку onphp.
Если некоторое количество людей подтвердит, что bc не ломается, тесты проходят, то я это смержу. Но на анализ кода у меня времени нет, к сожалению :(

@AlexeyDsov
Copy link
Member

Я жив, пользуюсь onphp. Но вот на анализ кода тоже боюсь что сейчас времени не найду.

@ssserj
Copy link
Member

ssserj commented Aug 4, 2014

Я жив. Компания пользуется onPHP, но уже ~2 года пишу только java.

Кстати, @dovg , я давно видел у вас в корпоративном репозитории тоже хороший код (кажется, вы там типы добавляли для postgresql), возможно, сейчас уже есть более интересные вещи, которые могут быть интересны сообществу и также можно влить в репозиторий?

Кстати, какая ветка сейчас считается активной. 1.0 или 1.1?
@AlexeyDsov какая?

@AlexeyDsov
Copy link
Member

@ssserj Активная считается ветка мастер. Это ни 1.1 и не 1.0.
У нас самих используется ветка с namespace'ами почти не отличающаяся от мастера.

@dovg
Copy link
Member

dovg commented Aug 4, 2014

@ssserj, да, там было много хорошего, но по некоторым причинам оно в апстрим не попало. Основная причина была в том, что никто не хотел уделять время написанию юнит-тестов и презентации кода в сообществе.

Я сейчас, к сожалению, не могу уделять время onphp, равно как и многие в этом чате :)
Может расширим число число участников https://github.com/orgs/onPHP/people ?

@glushchenko
Copy link

Форк с неймспейсами в паблике есть? У нас несколько больших проектов на onPHP, думаю мог бы уделить некоторое время.

@AlexeyDsov
Copy link
Member

@glushchenko в моём форке ветка masterNs. Плюс ещё есть от него ветка с парой коммитов добавляющий TaggableDaoWorker - masterNsTW и ещё одна ветка с конвертеров кода на неймспейсы masterNsConverter. Но последние две ветки от masterNs отличаются лишь парой коммитов.

@glushchenko
Copy link

Возьмите меня на испытательный срок, я хоть бы неймспейсы влил, совсем же грустно сейчас.

@dovg
Copy link
Member

dovg commented Aug 7, 2014

Только @ssserj может добавить.
@glushchenko, если очень хочется - готовь пулреквест, я волью.

@AlexeyDsov
Copy link
Member

Если речь про namespace'ы то я думаю не стоит их вливать. Kучше просто взять веточку, перенести в этот репозиторий и вести параллельно пока параллельно, т.к. всё таки раз и перейти на неймспейсы вряд ли кто-то может.
Если про текущий issue речь то да, для него бы реквест.

@glushchenko
Copy link

@dovg @AlexeyDsov согласен, ну так давайте соберём это всё в новую ветку. С меня пулл риквест, с вас новая ветка.

@AlexeyDsov
Copy link
Member

Дабы не обсуждать тему неймспейсов далее здесь, завёл отдельное обсуждение - #215

@pupkinV
Copy link
Contributor

pupkinV commented Nov 29, 2014

@glushchenko у вас что-нибудь получилось или я невнимательно смотрел пул-реквесты?

@glushchenko
Copy link

@pupkinV всё так и заглохло, в старой ветке без неймспейсов оно вроде и не нужно.

@ShadeSlider
Copy link
Author

@pupkinV Все забили, увы.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants