Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

FIX: updateCMSFields not accepting var by reference #284

Merged
merged 1 commit into from

3 participants

@mandrew

updateCMSFields should be taking the fields var by reference.
@see silverstripe/silverstripe-framework#1156

@chillu chillu merged commit 1525684 into silverstripe:3.0
@normann

@mandrew, OMG, finally got you come up again. Where are you, still in UK?

@normann haha yip mate still in the UK and things going ok. Might see you later in the year though ; )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 4, 2013
  1. @mandrew

    FIX: updateCMSFields not accepting var by reference

    mandrew authored
    updateCMSFields should be taking the fields var by reference.
    @see silverstripe/silverstripe-framework#1156
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  code/model/SiteTreeFileExtension.php
View
2  code/model/SiteTreeFileExtension.php
@@ -9,7 +9,7 @@ class SiteTreeFileExtension extends DataExtension {
'BackLinkTracking' => 'SiteTree'
);
- public function updateCMSFields(FieldList $fields) {
+ public function updateCMSFields(FieldList &$fields) {
$fields->insertAfter(new ReadonlyField('BackLinkCount',
_t('AssetTableField.BACKLINKCOUNT', 'Used on:'),
$this->BackLinkTracking()->Count() . ' ' . _t('AssetTableField.PAGES', 'page(s)')),
Something went wrong with that request. Please try again.