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

Fix debugging problem #27

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix debugging problem #27

wants to merge 4 commits into from

Conversation

ZoRDoK
Copy link

@ZoRDoK ZoRDoK commented May 4, 2013

There is a long story. You hear about problem with smarty "debugging". It's because in depth yii code renderPartial and render always set $return to "true". And always we have fetch() (not display()). And then always a debug window doesn't render.

If we call renderFile() — ok, we have debug window worked well, and smarty understand path like a "blog/index.tpl", but old code don't.

There is templateExists() — core Smarty function, included in both versions.

ZoRDoK added 3 commits May 4, 2013 20:59
There is a long story. You hear about problem with smarty "debugging". It's because in depth yii code renderPartial and render always set $return to "true". And always we have fetch() (not display()). And then always a debug window doesn't render.

If we call renderFile() — ok, we have debug window worked well, and smarty understand path like a "blog/index.tpl", but old code don't.

There is templateExists() — core Smarty function, included in both versions.
Add some stuff to docs
@samdark
Copy link
Member

samdark commented May 4, 2013

Looks good to me. Can you update changelog?

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

Successfully merging this pull request may close these issues.

2 participants