Tidying up output, removing blank lines

diff --git a/lib/NCIP/Handler.pm b/lib/NCIP/Handler.pm
index bf1d7c7..713967c 100644 (file)
--- a/lib/NCIP/Handler.pm
+++ b/lib/NCIP/Handler.pm
@@ -41,7 +41,11 @@ sub render_output {
 
     my $vars     = shift;
     my $template = Template->new(
-        { INCLUDE_PATH => $self->templates, } );
+        {
+            INCLUDE_PATH => $self->templates,
+            POST_CHOMP   => 1
+        }
+    );
     my $output;
     $template->process( $templatename, $vars, \$output );
     return $output;

diff --git a/lib/NCIP/Handler/LookupUser.pm b/lib/NCIP/Handler/LookupUser.pm
index d766122..891eedb 100644 (file)
--- a/lib/NCIP/Handler/LookupUser.pm
+++ b/lib/NCIP/Handler/LookupUser.pm
@@ -36,8 +36,13 @@ sub handle {
         my $user = NCIP::User->new(
             { userid => $user_id->textContent(), ils => $self->ils } );
         $user->initialise();
-        my $root = $xmldoc->documentElement();
+
+        # if we have blank user, we need to return that
+        # and can skip looking for elementtypes
+
+        my $root     = $xmldoc->documentElement();
         my @elements = $root->findnodes('LookupUser/UserElementType/Value');
+
         #set up the variables for our template
         my $vars;
         $vars->{'messagetype'} = 'LookupUserResponse';

diff --git a/templates/includes/LookupUserResponse.inc b/templates/includes/LookupUserResponse.inc
index ec03c40..f6ce676 100644 (file)
--- a/templates/includes/LookupUserResponse.inc
+++ b/templates/includes/LookupUserResponse.inc
@@ -1,3 +1,17 @@
+<UserOptionalFields>
 [% FOREACH element IN elements %]
-  [% element.textContent %]
+  [% IF element.textContent == 'User Address Information' %]
+    
+  [% END %]
+  [% IF element.textContent == 'Name Information' %]
+    <NameInformation>
+      <PersonalNameInformation>
+        <StructuredPersonalUserName>
+          <GivenName>[% user.userdata.firstname %]</GivenName>
+          <Surname>[% user.userdata.surname %]</Surname>
+        </StructuredPersonalUserName>
+      </PersonalNameInformation>
+    </NameInformation>
+  [% END %]
 [% END %]
+</UserOptionalFields/>