CHtml::cssFile() url fix 2 #1542

Merged
merged 1 commit into from Oct 6, 2012

Conversation

3 participants
Contributor

creocoder commented Oct 6, 2012

No description provided.

samdark merged commit a48a1cc into yiisoft:master Oct 6, 2012

Owner

cebe commented Oct 7, 2012

@creocoder @samdark whats the problem with this? a description would have been nice ;)

Also when I have apersands in my url they should be encoded as & I think. Why did you remove it?

Owner

samdark commented Oct 7, 2012

It's a fix for a fix that resulted in unit tests breakage.

Owner

cebe commented Oct 7, 2012

seen that (#1540). Re-checked and found that linkTag encodes attributes by default so encode here will result in double encoding. Everything is fine though.

Contributor

creocoder commented Oct 7, 2012

@cebe Yes, all fine there.

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