Skip to content

+ test for lazy encapsulant added #56

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Dec 2, 2011
Merged

+ test for lazy encapsulant added #56

merged 7 commits into from
Dec 2, 2011

Conversation

dovg
Copy link
Member

@dovg dovg commented Nov 28, 2011

Добавил тест, который ломает изменения в LiteMetaProperty, см issue #55

@dovg dovg mentioned this pull request Nov 28, 2011
Closed
@dovg
Copy link
Member Author

dovg commented Nov 29, 2011

Предлагаю в таком виде залить.

@AlexeyDsov
Copy link
Member

А зачем же теперь откатывать те изменения с которых все началось? Вроде ж все теперь поправили.
Это я про d25d800. Ну и тут уже написал комментарий

@dovg
Copy link
Member Author

dovg commented Nov 29, 2011

@AlexeyDsov сможешь подготовить "правильный" pull request, который проходил бы тесты? Достаточно откатить d25d800, или еще что-то надо делать?

@AlexeyDsov
Copy link
Member

Да, достаточно откатить d25d800

@dovg
Copy link
Member Author

dovg commented Nov 29, 2011

Вольем?

@AlexeyDsov
Copy link
Member

Тесты вроде б проходят. Во всяком случае у меня по разным базам все проходило, я за.

@dovg
Copy link
Member Author

dovg commented Nov 30, 2011

Принципиальный момент - это без пересбора меты ломает bc. В бранч 1.0 вливать будем?

@AlexeyDsov
Copy link
Member

Вроде бы время от времени такие фиксы уже бывали. С другой стороны не вечно же мержиться в 1.0... В общем тему по версионированию надо обсуждать

@ssserj
Copy link
Member

ssserj commented Dec 1, 2011

Давайте делать и в master и в 1.0.
Реально, то что мы "накопали" - это баг, 1.0. Этот фикс сделает 1.0. более стабильным и быстрым.

@dovg
Merge branch 'onphp-master' into metaPropertyBug
6264e9f 
* onphp-master:
  update doc/ChangeLog for onPHP#46
  * view test tune
  * view test initialization fix
  + add simple view render test

Conflicts:
	doc/ChangeLog
dovg added a commit that referenced this pull request Dec 2, 2011
@dovg
Merge pull request #56 from dovg/metaPropertyBug
554b118 
+ test for lazy encapsulant added.
+ new implementation of encapsulant getId added
@dovg dovg merged commit 554b118 into onPHP:master Dec 2, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dovg @AlexeyDsov @ssserj