From c8a371afff72055285a6db9729bda0bacb90f9e3 Mon Sep 17 00:00:00 2001 From: Silvio Date: Thu, 19 Aug 2010 17:50:42 -0300 Subject: Fixing audit --- classes/IsisAudit.php | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) (limited to 'classes/IsisAudit.php') 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"; + } } } } -- cgit v1.2.3