mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-16 03:42:41 +01:00
4682e0c104
Summary: I make this error quite often: I forget to declare a property I am writing to or I make a typo in it. PHP implicitly creates a public property which I don't like. I would much rather see a linter warning me against this than this runtime check but writing it is very difficult: - We need to explore all parents of the class we are checking. - It is even possible that children will declare that property but it's OK to treat this as error anyway. - We can extend also builtin or external classes. - It's somewhat doable for `$this` but even more complex for any `$obj` because we don't know the class of it. This should catch significant part of these errors and I'm fine with that. I don't plan escalating to exception because this error is not fatal and should not stop the application from working. Test Plan: Loaded homepage, checked log. Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D3601
57 lines
1.5 KiB
PHP
57 lines
1.5 KiB
PHP
<?php
|
|
|
|
/*
|
|
* Copyright 2012 Facebook, Inc.
|
|
*
|
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
|
* you may not use this file except in compliance with the License.
|
|
* You may obtain a copy of the License at
|
|
*
|
|
* http://www.apache.org/licenses/LICENSE-2.0
|
|
*
|
|
* Unless required by applicable law or agreed to in writing, software
|
|
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
* See the License for the specific language governing permissions and
|
|
* limitations under the License.
|
|
*/
|
|
|
|
abstract class AphrontView {
|
|
|
|
protected $children = array();
|
|
|
|
final public function appendChild($child) {
|
|
$this->children[] = $child;
|
|
return $this;
|
|
}
|
|
|
|
final protected function renderChildren() {
|
|
$out = array();
|
|
foreach ($this->children as $child) {
|
|
$out[] = $this->renderSingleView($child);
|
|
}
|
|
return implode('', $out);
|
|
}
|
|
|
|
final protected function renderSingleView($child) {
|
|
if ($child instanceof AphrontView) {
|
|
return $child->render();
|
|
} else if (is_array($child)) {
|
|
$out = array();
|
|
foreach ($child as $element) {
|
|
$out[] = $this->renderSingleView($element);
|
|
}
|
|
return implode('', $out);
|
|
} else {
|
|
return $child;
|
|
}
|
|
}
|
|
|
|
abstract public function render();
|
|
|
|
public function __set($name, $value) {
|
|
phlog('Wrote to undeclared property '.get_class($this).'::$'.$name.'.');
|
|
$this->$name = $value;
|
|
}
|
|
|
|
}
|