Skip to content
Snippets Groups Projects
Commit d5ade79d authored by Peter Rotich's avatar Peter Rotich
Browse files

Merge pull request #260 from greezybacon/issue/multi-fetch-same-user


fetch: Multiple email fetches may reflect same user

Reviewed-By: default avatarPeter Rotich <peter@osticket.com>
parents 17b9a28a 363bc4b8
No related branches found
No related tags found
No related merge requests found
......@@ -41,8 +41,8 @@ class DynamicForm extends VerySimpleModel {
var $_fields;
var $_dfields;
function getFields() {
if (!isset($this->_fields)) {
function getFields($cache=true) {
if (!isset($this->_fields) || !$cache) {
$this->_fields = array();
foreach ($this->getDynamicFields() as $f)
// TODO: Index by field name or id
......@@ -83,7 +83,7 @@ class DynamicForm extends VerySimpleModel {
function getForm($source=false) {
if (!$this->_form || $source) {
$fields = $this->getFields();
$fields = $this->getFields(false);
$this->_form = new Form($fields, $source, array(
'title'=>$this->title, 'instructions'=>$this->instructions));
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment