diff options
Diffstat (limited to 'classes')
| -rw-r--r-- | classes/IsisAudit.php | 24 | ||||
| -rw-r--r-- | classes/IsisFinder.php | 6 |
2 files changed, 13 insertions, 17 deletions
diff --git a/classes/IsisAudit.php b/classes/IsisAudit.php index 79cbe26..63446e1 100644 --- a/classes/IsisAudit.php +++ b/classes/IsisAudit.php @@ -6,31 +6,27 @@ class IsisAudit extends IsisFinder { /** * Run a standard audit procedure. - * - * @todo - * Test. */ public function run() { foreach ($this->format['fields'] as $field) { - $field_name = $this->getFieldName($field); - $repetition = $this->nextRepetition(null, $field_name); + $repetition = $this->nextRepetition($field); // Check for repetitions. - if ($field['repeat'] && !$repetition) - { - echo "Field $field_name is configured for repetitions but no repetitions found.\n"; + if ($field['repeat'] && !$repetition) { + echo "Field ". $field['name'] ." is configured for repetitions but no repetitions found.\n"; } elseif (!$field['repeat'] && $repetition) { - echo "Field $field_name is not configured for repetitions but a repetition was found.\n"; + echo "Field ". $field['name'] ." is not configured for repetitions but a repetition was found for entry ". $repetition[0] .".\n"; } // Check for subfields. - foreach ($field['subfields'] as $subfield) { - $subfield_name = $this->getSubfieldName($field, $subfield); - $next_subfield = $isis->nextSubfield(null, $field_name, $subfield_name); + if (isset($field['subfields'])) { + foreach ($field['subfields'] as $subfield) { + $next_subfield = $this->nextSubfield($field, $subfield); - if (!$next_subfield) { - echo "No occurrences found for field $field_name and subfield $subfield_name\n"; + if (!$next_subfield) { + echo "No occurrences found for field ". $field['name'] ." and subfield $subfield\n"; + } } } } diff --git a/classes/IsisFinder.php b/classes/IsisFinder.php index 26430c6..7f7405d 100644 --- a/classes/IsisFinder.php +++ b/classes/IsisFinder.php @@ -16,7 +16,7 @@ class IsisFinder extends IsisConnector { * @return * Next repetition entry and result. */ - public function nextRepetition($entry = 1, $field) { + public function nextRepetition($field, $entry = 1) { foreach(new IsisEntryIterator($this, $entry) as $entry => $result) { if (count($this->getValues($field)) >= 2) { return array($entry, $result); @@ -38,7 +38,7 @@ class IsisFinder extends IsisConnector { * @return * Next occurrence. */ - public function nextField($entry = 1, $field) { + public function nextField($field, $entry = 1) { foreach(new IsisEntryIterator($this, $entry) as $entry => $result) { if (count($this->getValues($field)) > 0) { return array($entry, $result); @@ -63,7 +63,7 @@ class IsisFinder extends IsisConnector { * @return * Next occurrence. */ - public function nextSubfield($entry = 1, $field, $subfield) { + public function nextSubfield($field, $subfield, $entry = 1) { foreach(new IsisEntryIterator($this, $entry) as $entry => $result) { if ($this->hasSubfieldInRows($field, $subfield) !== FALSE) { return array($entry, $result); |