From c8a371afff72055285a6db9729bda0bacb90f9e3 Mon Sep 17 00:00:00 2001
From: Silvio <silvio@devlet.com.br>
Date: Thu, 19 Aug 2010 17:50:42 -0300
Subject: Fixing audit

---
 classes/IsisAudit.php  | 24 ++++++++++--------------
 classes/IsisFinder.php |  6 +++---
 2 files changed, 13 insertions(+), 17 deletions(-)

(limited to 'classes')

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);
-- 
cgit v1.2.3