The expression $row of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.
This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an
empty array is considered to be equal (but not identical) to false, this is not always apparent.
Consider making the comparison explicit by using empty(..) or !empty(...) instead.
Loading history...
49
if ($row->guid == $this->attributes['guid']) {
50
// can save active site
51
return parent::save();
52
}
53
54
_elgg_services()->logger->error('More than 1 site entity cannot be created.');
55
return false;
56
}
57
58
2
return parent::save(); // TODO: Change the autogenerated stub
59
}
60
61
/**
62
* Delete the site.
63
*
64
* @note You cannot delete the current site.
65
*
66
* @param bool $recursive If true (default) then all entities which are owned or contained by $this will also be deleted.
67
*
68
* @return bool
69
* @throws SecurityException
70
*/
71
public function delete($recursive = true) {
72
if ($this->guid == 1) {
73
throw new \SecurityException('You cannot delete the current site');
74
}
75
76
return parent::delete($recursive);
77
}
78
79
/**
80
* Disable the site
81
*
82
* @note You cannot disable the current site.
83
*
84
* @param string $reason Optional reason for disabling
85
* @param bool $recursive Recursively disable all contained entities?
86
*
87
* @return bool
88
* @throws SecurityException
89
*/
90
public function disable($reason = "", $recursive = true) {
91
if ($this->guid == 1) {
92
throw new \SecurityException('You cannot disable the current site');
93
}
94
95
return parent::disable($reason, $recursive);
96
}
97
98
/**
99
* {@inheritdoc}
100
*/
101
2
public function __set($name, $value) {
102
2
if ($name === 'url') {
103
_elgg_services()->logger->warn("ElggSite::url cannot be set");
104
return;
105
}
106
2
parent::__set($name, $value);
107
2
}
108
109
/**
110
* {@inheritdoc}
111
*/
112
4976
public function __get($name) {
113
4976
if ($name === 'url') {
114
1
return $this->getURL();
115
}
116
4976
return parent::__get($name);
117
}
118
119
/**
120
* Returns the URL for this site
121
*
122
* @return string The URL
123
*/
124
3
public function getURL() {
125
3
return _elgg_config()->wwwroot;
126
}
127
128
/**
129
* {@inheritdoc}
130
*/
131
protected function prepareObject($object) {
132
$object = parent::prepareObject($object);
133
$object->name = $this->getDisplayName();
134
$object->description = $this->description;
135
unset($object->read_access);
136
return $object;
137
}
138
139
/**
140
* Get the domain for this site
141
*
142
* @return string
143
* @since 1.9
144
*/
145
1
public function getDomain() {
146
1
$breakdown = parse_url($this->url);
147
1
return $breakdown['host'];
148
}
149
150
/**
151
* Get the email address for the site
152
*
153
* This can be set in the basic site settings or fallback to noreply@domain
Elgg /
Elgg
Steve Clay
This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.
Consider making the comparison explicit by using
empty(..)or! empty(...)instead.