Тянет ли на баг реквест

[Redjik]

В AR Yii, есть метод count
он не запускает метод beforeFind
все остальные query на вытаскивание данных это делают

у себя поправил это

 

[флоппик]

Почему не тянет? Пость

 

[Redjik]

http://code.google.com/p/yii/issues/detail?id=764

хм, так и не решили ... странно.

 

[Ragazzo]

Иван Redjik Матвеев
Могли забыть, Sam Dark как-то поправил баг тут, который был связан с тем, что в beforFind() нельзя было достать criteria, просто забывают некоторые баги.

 

[AmdY]

addCondition('deleted' != 1) - ???

 

[Ragazzo]

AmdY
Неверный синтаксис, видимо просто для примера, ну там кавычек не хватает или про что ты?

 

[Вурдалак]

Да это просто новый уровень. Если Symfony парсит doc-блоки, то Yii по бектрейсу парсит кусок PHP-кода, из которого он был вызван.

 

[grigori]

не забыли, Саша написал: создайте отдельный репорт, но, видимо, никому особо не надо

 

[Sam Dark]

Закидывайте на github. Если так и останется тут в топике — точно забудем.

 

[Redjik]

закину, но попзже, еще не разобрался как phpStrom к github подвязать

 

[Redjik]

Сделал pull реквест, правда накосячил =(
mercurial умеет отличать строчки кода, которые изменились, даже если делать полный копи-паст...

 

[Вурдалак]

А кто не умеет?

 

[AmdY]

Иван Redjik Матвеев
перевод строк?

 

[Redjik]

Да в топку это, унылый гит - два раза делал push оба раза он мне показывал изменения - все удалено, а ниже все добавлено.
Смерджить не могу, ибо получается весь код будто переписан.

 

[Redjik]

Постараюсь без батхерта, но уже психи изрядные словил...

phpStorm показывает только те изменения, которые сделал я.
тоесть буквально 40-50 строчек кода+комментариев

коммит проходит нормально, все изменения видны

но как только делаю пуш, эта сука вот что выдает https://github.com/Redjik/yii/commit/dae0922916c9eb38741be62338bf33b3e911ad4f

 

[Вурдалак]

Ну, пробелы/табуляция, наверное. У тебя phpStorm видимо без учёта пробельных символов diff показывает. Вообще, полезно с консолькой работать, а не через IDE.

 

[Redjik]

по сслыке видно, что там не пробелы и табцляция, там весь код переписывается

захотел блин доброе дело сделать...

 

[Andkorol]

Мб различия в Line endings?

 

[Вурдалак]

по сслыке видно, что там не пробелы и табцляция

Как ты это определил? Мне вот ничего не видно.

Вот хотя бы это что:

+      'onAfterFind'=>'afterFind',
+            'onBeforeCount'=>'beforeCount'	
+    ));

?

 

[Redjik]

ладно тут правда косяк, но я дописал ТОЛЬКО

'onBeforeCount'=>'beforeCount'

а он пишет что весь код, попробую башем сделать завтра