route assemble complete #26

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

zxcvdavid commented Dec 25, 2012

route assemble complete

Owner

laruence commented Dec 26, 2012

正在review中, 需要改写一些(主要是一些内存泄露, 和逻辑调优). 回头我改写完了, 发给你review下.

tests/071.phpt
+ )
+);
+--EXPECTF--
+string(58) "/yafmodule/yafcontroller/yafaction/tkey1/tval1/tkey2/tval2"
@laruence

laruence Mar 1, 2013

Owner

这个static有问题啊
之前的所有的reverse
第二个参数都变成了query_string
而这个static, 却变成了URI
不一致啊

Contributor

zxcvdavid commented May 6, 2013

@laruence 鸟哥,把这个版本仍然存在问题的地方列到这里吧,我老忘。。

Merge branch 'master' of https://github.com/laruence/php-yaf
Conflicts:
	tests/068.phpt
	tests/069.phpt
Contributor

zxcvdavid commented Sep 9, 2013

write a new one

@zxcvdavid zxcvdavid closed this Sep 9, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment