Update toString() to use late static binding for encoding methods #3960

Merged
merged 1 commit into from Mar 11, 2013

3 participants

@jvandemo

Updated the toString() method in \Zend\Uri\Uri so it uses late static binding to call the encoding methods.

Replaced self:: calls with static:: calls in toString().

This resolves problems when writing your own custom Uri classes.

Change-Id: I470379b8527852d5f36145c88ab05cbb51582103

Jurgen Van de Moere Update toString() to use late static binding for encoding methods
Change-Id: I470379b8527852d5f36145c88ab05cbb51582103
e9f45f1
@ezimuel ezimuel was assigned Mar 8, 2013
@weierophinney weierophinney added a commit that referenced this pull request Mar 11, 2013
@weierophinney weierophinney Merge branch 'hotfix/3960' into develop
Forward port #3960
dfe1dd4
@weierophinney weierophinney added a commit that referenced this pull request Mar 11, 2013
@weierophinney weierophinney Merge branch 'hotfix/3960'
Clse #3960
5d1efb6
@weierophinney weierophinney merged commit e9f45f1 into zendframework:master Mar 11, 2013

1 check failed

Details default The Travis build failed
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/3960'
Clse #3960
485d47c
@ghost Unknown pushed a commit that referenced this pull request Jul 14, 2013
@weierophinney weierophinney Merge branch 'hotfix/3960' into develop
Forward port #3960
ce948ab
@weierophinney weierophinney added a commit to zendframework/zend-uri that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge pull request zendframework/zendframework#3960 from jvandemo/hot…
…fix/zend-uri-late-static-binding

Update toString() to use late static binding for encoding methods
74f27f1
@weierophinney weierophinney added a commit to zendframework/zend-uri that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3960' 7dfef7f
@weierophinney weierophinney added a commit to zendframework/zend-uri that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/3960' into develop 99ff9de
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment