Attempt re-login
[pazpar2-moved-to-github.git] / www / iphone / example_client.js
index 6eacd1b..3dcd63d 100644 (file)
@@ -4,31 +4,44 @@
 // create a parameters array and pass it to the pz2's constructor
 // then register the form submit event with the pz2.search function
 // autoInit is set to true on default
-var usesessions = true;
-var pazpar2path = '/pazpar2/search.pz2';
+var usesessions = false;
+var pazpar2path = '/service-proxy/';
 var showResponseType = '';
+// Facet configuration
 var querys = {'su': '', 'au': '', 'xt': ''};
+var query_client_server = {'su': 'subject', 'au': 'author', 'xt': 'xtargets'};
+var querys_server = {};
+var useLimit = 1;
+// Fail to get JSON working stabil.
+var showResponseType = 'xml';
 
-if (document.location.hash == '#useproxy' || document.location.search.match("useproxy=true")) {
-    usesessions = false;
-    pazpar2path = '/service-proxy/';
+var imageHelper = new ImageHelper();
+
+if (document.location.hash == '#pazpar2' || document.location.search.match("useproxy=false")) {
+    usesessions = true;
+    pazpar2path = '/pazpar2/search.pz2';
     showResponseType = 'xml';
 }
 
 my_paz = new pz2( { "onshow": my_onshow,
-                    "showtime": 500,            //each timer (show, stat, term, bytarget) can be specified this way
+//                    "showtime": 2000,            //each timer (show, stat, term, bytarget) can be specified this way
                     "pazpar2path": pazpar2path,
                     "oninit": my_oninit,
-                    "onstat": my_onstat,
+                    "onstat": null,
                     "onterm": my_onterm_iphone,
                     "termlist": "xtargets,subject,author",
-                    "onbytarget": my_onbytarget,
+                    "onbytarget": null,
                    "usesessions" : usesessions,
                     "showResponseType": showResponseType,
-                    "onrecord": my_onrecord } );
+                    "onrecord": my_onrecord,
+                   "errorhandler" : my_onerror} 
+);
 // some state vars
 var curPage = 1;
-var recPerPage = 20;
+var recPerPage = 10;
+var recToShowPageSize = 20;
+var recToShow = recToShowPageSize;
+var recIDs = {};
 var totalRec = 0;
 var curDetRecId = '';
 var curDetRecData = null;
@@ -43,6 +56,29 @@ var tab = "recordview";
 var triedPass = "";
 var triedUser = "";
 
+//
+// pz2.js event handlers:
+//
+function my_onerror(error) {
+    switch(error.code) {
+        // No targets!
+    case "8": alert("No resources were selected for the search"); break;
+       // target not configured, this is a pazpar2 bug
+        // but for now simply research
+    case "9": 
+       triggerSearch(); 
+       break;
+        // authentication
+    case "100" : 
+       loginFormSubmit();
+       //window.location = "login.html"; 
+       break;
+    default: 
+       alert("Unhandled error: " + error.code);
+       throw error; // display error in JavaScript console
+    }
+}
+
 function loginFormSubmit() {
     triedUser = document.loginForm.username.value;
     triedPass = document.loginForm.password.value;
@@ -51,6 +87,26 @@ function loginFormSubmit() {
        authCb, authCb);
 }
 
+function handleKeyPress(e)  
+{  
+  var key;  
+  if(window.event)  
+    key = window.event.keyCode;  
+  else  
+    key = e.which;  
+
+  if(key == 13 || key == 10)  
+  {  
+      button = document.getElementById('button');
+      button.focus();
+      button.click();
+
+      return false;  
+  }  
+  else  
+    return true;  
+}  
+
 function authCb(authData) {
     if (!authData.loginFailed) {
        triedUser = "";
@@ -86,19 +142,19 @@ function logout() {
 function logInOrOut() {
     var loginElement = document.getElementById("login");
     if (loginElement.innerHTML == 'Login')
-       login();
+       login();
     else
-       logout();
+       logout();
 }
 function loggedIn() {
     var login = document.getElementById("login");
     login.innerHTML = 'Logout';
-    //    login.onclick = 'logout();';
-    domReady();
+    document.getElementById("log").innerHTML = login.innerHTML;
 }
 
 function auth_check() {
-    auth.check(loggedIn, login, true);
+    auth.check(loggedIn, login);
+    domReady();
 }
 
 //
@@ -109,38 +165,138 @@ function my_oninit() {
     my_paz.bytarget();
 }
 
+function showMoreRecords() {
+    var i = recToShow;
+    recToShow += recToShowPageSize;
+    for ( ; i < recToShow && i < recPerPage; i++) {
+       var element = document.getElementById(recIDs[i]);
+       if (element)
+           element.style.display = '';
+    }
+    if (i == recPerPage) {
+       var element = document.getElementById('recdiv_END');
+       if (element)
+           element.style.display = 'none';
+    }
+}
+
+function hideRecords() {
+    for ( var i = 0; i < recToShow; i++) {
+       var element = document.getElementById(recIDs[i]);
+       if (element && recIDs != curDetRecId)
+           element.style.display = 'none';
+    }
+    var element = document.getElementById('recdiv_END');
+    if (element)
+       element.style.display = 'none';
+}
+
+function showRecords() {
+    for (var i = 0 ; i < recToShow && i < recPerPage; i++) {
+       var element = document.getElementById(recIDs[i]);
+       if (element)
+           element.style.display = '';
+    }
+    var element = document.getElementById('recdiv_END');
+    if (element) {
+       if (i == recPerPage)
+           element.style.display = 'none';
+       else
+           element.style.display = '';
+    }
+}
+
+
+
+
 function my_onshow(data) {
     totalRec = data.merged;
     // move it out
     var pager = document.getElementById("pager");
     pager.innerHTML = "";
-    pager.innerHTML +='<hr/><div style="float: right">Displaying: ' 
+    drawPager(pager);
+    pager.innerHTML +='<div class="status">Displaying: ' 
                     + (data.start + 1) + ' to ' + (data.start + data.num) +
                      ' of ' + data.merged + ' (found: ' 
                      + data.total + ')</div>';
-    drawPager(pager);
 
     var results = document.getElementById("results");
-  
+    
     var html = [];
+    if (data.hits == undefined) 
+       return ;
+    var style = '';
     for (var i = 0; i < data.hits.length; i++) {
         var hit = data.hits[i];
-             html.push('<li id="recdiv_'+hit.recid+'" >'
-           /* +'<span>'+ (i + 1 + recPerPage * (curPage - 1)) +'. </span>' */
-            +'<a href="#" id="rec_'+hit.recid
-            +'" onclick="showDetails(this.id);return false;">' 
-            + hit["md-title"] +'</a> '); 
-             if (hit["md-title-responsibility"] !== undefined) {
-           html.push('<a href="#">'+hit["md-title-responsibility"]+'</a> ');
-             if (hit["md-title-remainder"] !== undefined) {
-               html.push('<a href="#">' + hit["md-title-remainder"] + ' </a> ');
-             }
-       }
+       var recDivID = "recdiv_" + hit.recid; 
+       recIDs[i] = recDivID;
+       var lines = 0;
+       if (i == recToShow)
+           style = ' style="display:none" ';
+       html.push('<li class="img arrow" id="' + recDivID + '" ' + style +  '>' );
+       html.push('<a class="img" href="#' + i + '" id="rec_'+hit.recid + '" onclick="showDetails(this.id);return false;" >');
+       if (1) {
+            var useThumbnails = hit["md-use_thumbnails"];
+            var thumburls = hit["md-thumburl"];
+            if (thumburls && (useThumbnails == undefined || useThumbnails == "1")) {
+               var thumbnailtag = imageHelper.getImageTagByRecId(hit.recid,"md-thumburl", 60, "S"); 
+               html.push(thumbnailtag);
+           } else { 
+               if (hit["md-isbn"] != undefined) { 
+                   var coverimagetag = imageHelper.getImageTagByRecId(hit.recid, "md-isbn", 60, "S"); 
+                   if (coverimagetag.length>0) { 
+                        html.push(coverimagetag);
+                   } else { 
+                       html.push("&nbsp;");
+                   }
+               }
+           }
+       } 
+       html.push("</a>");
+       html.push('<a href="#" id="rec_'+hit.recid + '" onclick="showDetails(this.id);return false;">');
+       html.push(hit["md-title"]); 
+       html.push("</a>");
+
+       if (hit["md-title-remainder"] != undefined) {
+           html.push('<a href="#" id="rec_'+hit.recid + '" onclick="showDetails(this.id);return false;">');
+           html.push(hit["md-title-remainder"]);
+           html.push("</a>");
+           lines++;
+       }
+       if (hit["md-author"] != undefined) {
+           html.push('<a href="#" id="rec_'+hit.recid + '" onclick="showDetails(this.id);return false;">');
+           html.push(hit["md-author"]);
+           html.push("</a>");
+           lines++;
+       }
+       else if (hit["md-title-responsibility"] != undefined) {
+           html.push('<a href="#" id="rec_'+hit.recid + '" onclick="showDetails(this.id);return false;">');
+           html.push(hit["md-title-responsibility"]);
+           html.push("</a>");
+           lines++;
+       }
+       for (var idx = lines ; idx < 2 ; idx++) {
+           html.push('<a href="#" id="rec_'+hit.recid + '" onclick="showDetails(this.id);return false;">');
+           html.push("&nbsp;");            
+           html.push("</a>");
+       }
+/*
         if (hit.recid == curDetRecId) {
             html.push(renderDetails_iphone(curDetRecData));
         }
-       html.push('</div>');
+*/
+       html.push('</li>');
     }
+    if (data.activeclients == 0)
+       document.getElementById("loading").style.display = 'none';
+/*
+    // set up "More..." if needed. 
+    style = 'display:none';
+    if (recToShow < recPerPage) {
+       style = 'display:block';
+    }
+    html.push('<li class="img" id="recdiv_END" style="' + style + '"><a onclick="showMoreRecords()">More...</a></li>');     
+*/
     replaceHtml(results, html.join(''));
 }
 
@@ -156,13 +312,15 @@ function my_onstat(data) {
                         + '/' + data.hits + ' :.</span>';
 }
 
-function showhide(newtab) {
+function showhide(newtab, hash) {
     var showtermlist = false;
     if (newtab != null)
        tab = newtab;
     
     if (tab == "recordview") {
        document.getElementById("recordview").style.display = '';
+       if (hash != undefined)
+           document.location.hash = hash;
     }
     else 
        document.getElementById("recordview").style.display = 'none';
@@ -188,6 +346,16 @@ function showhide(newtab) {
     else
        document.getElementById("term_authors").style.display = 'none';
 
+    if (tab == "detailview") {
+       document.getElementById("detailview").style.display = '';
+    }
+    else {
+       document.getElementById("detailview").style.display = 'none';
+       var element = document.getElementById("rec_" + curDetRecId);
+       if (element != undefined)
+           element.scrollIntoView();
+
+    }
     if (showtermlist == false) 
        document.getElementById("termlist").style.display = 'none';
     else 
@@ -207,10 +375,10 @@ function my_onterm(data) {
     
     termlists.push('<div id="term_xtargets" >');
     termlists.push('<h4 class="termtitle">Sources</h4>');
-    termlists.push('<ul>');
-    termlists.push('<li><a href="#" target_id="reset_xt" onclick="limitOrResetTarget(\'reset_xt\',\'All\');return false;">All</a></li>');
+    termlists.push('<ul class="termlist">');
+    termlists.push('<li> <a href="#" target_id="reset_xt" onclick="limitOrResetTarget(\'reset_xt\',\'All\');return false;">All</a></li>');
     for (var i = 0; i < data.xtargets.length && i < SourceMax; i++ ) {
-        termlists.push('<li><a href="#" target_id='+data.xtargets[i].id
+        termlists.push('<li class="termlist"><a href="#" target_id='+data.xtargets[i].id
             + ' onclick="limitOrResetTarget(this.getAttribute(\'target_id\'), \'' + data.xtargets[i].name + '\');return false;">' 
            + data.xtargets[i].name + ' (' + data.xtargets[i].freq + ')</a></li>');
     }
@@ -219,7 +387,7 @@ function my_onterm(data) {
      
     termlists.push('<div id="term_subjects" >');
     termlists.push('<h4>Subjects</h4>');
-    termlists.push('<ul>');
+    termlists.push('<ul class="termlist">');
     termlists.push('<li><a href="#" target_id="reset_su" onclick="limitOrResetQuery(\'reset_su\',\'All\');return false;">All</a></li>');
     for (var i = 0; i < data.subject.length && i < SubjectMax; i++ ) {
         termlists.push('<li><a href="#" onclick="limitOrResetQuery(\'su\', \'' + data.subject[i].name + '\');return false;">' 
@@ -230,10 +398,10 @@ function my_onterm(data) {
             
     termlists.push('<div id="term_authors" >');
     termlists.push('<h4 class="termtitle">Authors</h4>');
-    termlists.push('<ul>');
-    termlists.push('<li><a href="#" onclick="limitOrResetQuery(\'reset_au\',\'All\');return false;">All<a></li>');
+    termlists.push('<ul class="termlist">');
+    termlists.push('<li><a href="#" onclick="limitOrResetQuery(\'reset_au\',\'All\');return false;">All</a></li>');
     for (var i = 0; i < data.author.length && i < AuthorMax; i++ ) {
-        termlists.push('<li><a href="#" onclick="limitQuery(\'au\', \'' + data.author[i].name +'\');return false;">' 
+        termlists.push('<li><a href="#" onclick="limitOrResetQuery(\'au\', \'' + data.author[i].name +'\');return false;">' 
                             + data.author[i].name 
                             + '  (' 
                             + data.author[i].freq 
@@ -246,27 +414,6 @@ function my_onterm(data) {
     showhide();
 }
 
-function serialize(array) {
-       var t = typeof (obj);
-       if (t != "object" || obj === null) {
-               // simple data type
-               return String(obj);
-       } else {
-               // recurse array or object
-               var n, v, json = [], arr = (obj && obj.constructor == Array);
-               for (n in obj) {
-                       v = obj[n];
-                       t = typeof (v);
-                       if (t == "string")
-                               v = '"' + v + '"';
-                       else if (t == "object" && v !== null)
-                               v = JSON.stringify(v);
-                       json.push((arr ? "" : '"' + n + '":') + String(v));
-               }
-               return (arr ? "" : "") + String(json) + (arr ? "]" : "}");
-       }
-}
-
 var termlist = {};
 function my_onterm_iphone(data) {
     my_onterm(data);
@@ -287,7 +434,6 @@ function my_onterm_iphone(data) {
         authors = authors + "au" + "|" + data.author[i].name + "|" + data.author[i].freq + "\n";
     }
     termlist["authors"] = authors;
-    //document.getElementById("log").innerHTML = targets + "\n" + subjects + "\n" + authors;
     callback.send("termlist", "refresh");
 }
 
@@ -304,15 +450,18 @@ function getAuthors() {
 }
 
 function my_onrecord(data) {
+
     // FIXME: record is async!!
     clearTimeout(my_paz.recordTimer);
     // in case on_show was faster to redraw element
-    var detRecordDiv = document.getElementById('det_'+data.recid);
-    if (detRecordDiv) return;
+    var detailRecordDiv = document.getElementById('detailrecord');
+    if (!detailRecordDiv) 
+       return;
     curDetRecData = data;
-    var recordDiv = document.getElementById('recdiv_'+curDetRecData.recid);
     var html = renderDetails_iphone(curDetRecData);
-    recordDiv.innerHTML += html;
+    detailRecordDiv.innerHTML = html;
+    showhide('detailview');
+    document.getElementById("loading").style.display = 'none';
 }
 
 function my_onrecord_iphone(data) {
@@ -352,22 +501,32 @@ function domReady ()
        applicationMode(true);
     else
        applicationMode(false);
+
+    var params = parseQueryString(window.location.search);
+    if (params.query) {
+       document.search.query.value = params.query;
+       onFormSubmitEventHandler();
+    }
 }
  
 function applicationMode(newmode) 
 {
-       var searchdiv = document.getElementById("searchForm");
-       if (newmode)
-               inApp = newmode;
-       if (inApp) {
+    var searchdiv = document.getElementById("searchForm");
+    if (newmode)
+       inApp = newmode;
+    if (inApp) {
        document.getElementById("heading").style.display="none";
                searchdiv.style.display = 'none';
-       }
-       else { 
-               searchdiv.style.display = '';
-               document.search.onsubmit = onFormSubmit;
-       }
-       callback.init();
+    }
+    else { 
+       
+       document.getElementById("nav").style.display="";
+       document.getElementById("normal").style.display="inline";
+       document.getElementById("normal").style.visibility="";
+       searchdiv.style.display = '';
+       document.search.onsubmit = onFormSubmit;
+    }
+    callback.init();
 }
 // when search button pressed
 function onFormSubmitEventHandler() 
@@ -395,9 +554,27 @@ function resetPage()
     totalRec = 0;
 }
 
+function getFacets() {
+    var result = "";
+    for (var key in querys_server) {
+       if (result.length > 0)
+           result += ","
+       result += querys_server[key];
+    }
+    return result;
+}
+
 function triggerSearch ()
 {
-    my_paz.search(document.search.query.value, recPerPage, curSort, curFilter);
+    // Restore to initial page size
+    recToShow = recToShowPageSize;
+    document.getElementById("loading").style.display = 'inline';
+    my_paz.search(document.search.query.value, recPerPage, curSort, curFilter, undefined,
+       {
+          "limit" : getFacets() 
+       }
+       );
+    showhide("recordview");
 }
 
 function loadSelect ()
@@ -409,34 +586,78 @@ function loadSelect ()
 // limit the query after clicking the facet
 function limitQuery(field, value)
 {
-       var newQuery = ' and ' + field + '="' + value + '"';
-       querys[field] += newQuery;
-    document.search.query.value += newQuery;
-    onFormSubmitEventHandler();
-    showhide("recordview");
+  var newQuery = ' and ' + field + '="' + value + '"';
+  querys[field] += newQuery;
+  document.search.query.value += newQuery;
+  onFormSubmitEventHandler();
+  showhide("recordview");
 }
 
-//limit the query after clicking the facet
+// limit the query after clicking the facet
+function limitQueryServer(field, value)
+{
+    // Check for client field usage
+    var fieldname = query_client_server[field];
+    if (!fieldname) 
+       fieldname = field;      
+    
+    var newQuery = fieldname + '=' + value.replace(",", "\\,").replace("|", "\\,");
+    // Does it already exists?
+    if (querys_server[fieldname]) 
+       querys_server[fieldname] += "," + newQuery;
+    else
+       querys_server[fieldname] = newQuery;
+//  document.search.query.value += newQuery;
+  onFormSubmitEventHandler();
+  showhide("recordview");
+}
+
+
+
+// limit the query after clicking the facet
 function removeQuery (field, value) {
        document.search.query.value.replace(' and ' + field + '="' + value + '"', '');
     onFormSubmitEventHandler();
     showhide("recordview");
 }
 
-//limit the query after clicking the facet
+// limit the query after clicking the facet
 function limitOrResetQuery (field, value, selected) {
-       if (field == 'reset_su' || field == 'reset_au') {
-               var reset_field = field.substring(6);
-               document.search.query.value = document.search.query.value.replace(querys[reset_field], '');
-               querys[reset_field] = '';
-           onFormSubmitEventHandler();
-           showhide("recordview");
-       }
-       else 
-               limitQuery(field, value);
+    if (useLimit) {
+       limitOrResetQueryServer(field,value, selected);
+       return ;
+    }
+    if (field == 'reset_su' || field == 'reset_au') {
+       var reset_field = field.substring(6);
+       document.search.query.value = document.search.query.value.replace(querys[reset_field], '');
+       querys[reset_field] = '';
+       onFormSubmitEventHandler();
+       showhide("recordview");
+    }
+    else 
+       limitQuery(field, value);
+       //alert("limitOrResetQuerry: query after: " + document.search.query.value);
+}
+
+// limit the query after clicking the facet
+function limitOrResetQueryServer (field, value, selected) {
+    if (field.substring(0,6) == 'reset_') {
+       var clientname = field.substring(6);
+       var fieldname = query_client_server[clientname];
+       if (!fieldname) 
+           fieldname = clientname;     
+       delete querys_server[fieldname];
+       onFormSubmitEventHandler();
+       showhide("recordview");
+    }
+    else 
+       limitQueryServer(field, value);
        //alert("limitOrResetQuerry: query after: " + document.search.query.value);
 }
 
+
+
+
 // limit by target functions
 function limitTarget (id, name)
 {
@@ -444,7 +665,6 @@ function limitTarget (id, name)
     resetPage();
     loadSelect();
     triggerSearch();
-    showhide("recordview");
     return false;
 }
 
@@ -469,7 +689,7 @@ function limitOrResetTarget(id, name) {
 function drawPager (pagerDiv)
 {
     //client indexes pages from 1 but pz2 from 0
-    var onsides = 6;
+    var onsides = 2;
     var pages = Math.ceil(totalRec / recPerPage);
     
     var firstClkbl = ( curPage - onsides > 0 ) 
@@ -480,10 +700,10 @@ function drawPager (pagerDiv)
         ? firstClkbl + 2*onsides
         : pages;
 
-    var prev = '<span id="prev">&#60;&#60; Prev</span><b> | </b>';
+    var prev = '<span id="prev">Prev</span><b> | </b>';
     if (curPage > 1)
         var prev = '<a href="#" id="prev" onclick="pagerPrev();">'
-        +'&#60;&#60; Prev</a><b> | </b>';
+        +'Prev</a><b> | </b>';
 
     var middle = '';
     for(var i = firstClkbl; i <= lastClkbl; i++) {
@@ -495,10 +715,10 @@ function drawPager (pagerDiv)
             + numLabel + ' </a>';
     }
     
-    var next = '<b> | </b><span id="next">Next &#62;&#62;</span>';
+    var next = '<b> | </b><span id="next">Next</span>';
     if (pages - curPage > 0)
     var next = '<b> | </b><a href="#" id="next" onclick="pagerNext()">'
-        +'Next &#62;&#62;</a>';
+        +'Next</a>';
 
     predots = '';
     if (firstClkbl > 1)
@@ -508,8 +728,8 @@ function drawPager (pagerDiv)
     if (lastClkbl < pages)
         postdots = '...';
 
-    pagerDiv.innerHTML += '<div style="float: none">' 
-        + prev + predots + middle + postdots + next + '</div><hr/>';
+    pagerDiv.innerHTML += '<div class="pager">' 
+        + prev + predots + middle + postdots + next + '</div>';
 }
 
 function showPage (pageNum)
@@ -559,20 +779,13 @@ function showDetails (prefixRecId) {
     var oldRecId = curDetRecId;
     curDetRecId = recId;
     
-    // remove current detailed view if any
-    var detRecordDiv = document.getElementById('det_'+oldRecId);
-    //alert("oldRecId: " + oldRecId + " " + detRecordDiv != null); 
-    // lovin DOM!
-    if (detRecordDiv)
-      detRecordDiv.parentNode.removeChild(detRecordDiv);
-
-    // if the same clicked, just hide
+    // if the same clicked, just show it again
     if (recId == oldRecId) {
-        curDetRecId = '';
-        curDetRecData = null;
+       showhide('detailview');
         return;
     }
     // request the record
+    document.getElementById("loading").style.display = 'inline';
     my_paz.record(recId);
 }
 
@@ -596,13 +809,13 @@ function renderDetails(data, marker)
     if (marker) details += '<tr><td>'+ marker + '</td></tr>';
     if (data["md-title"] != undefined) {
         details += '<tr><td><b>Title</b></td><td><b>:</b> '+data["md-title"];
-       if (data["md-title-remainder"] !== undefined) {
+       if (data["md-title-remainder"] != undefined) {
              details += ' : <span>' + data["md-title-remainder"] + ' </span>';
        }
-       if (data["md-title-responsibility"] !== undefined) {
-             details += ' <span><i>'+ data["md-title-responsibility"] +'</i></span>';
+       if (data["md-author"] != undefined) {
+             details += ' <span><i>'+ data["md-auhtor"] +'</i></span>';
        }
-         details += '</td></tr>';
+       details += '</td></tr>';
     }
     if (data["md-date"] != undefined)
         details += '<tr><td><b>Date</b></td><td><b>:</b> ' + data["md-date"] + '</td></tr>';
@@ -618,15 +831,45 @@ function renderDetails(data, marker)
     return details;
 }
 
-function renderLine(title, value) {
+function renderLine(title, value, tag) {
+    if (tag == undefined)
+       tag = 'big';
     if (value != undefined)
-        return '<li><h3>' + title + '</h3> <big>' + value + '</big></li>';
+        return '<li><h3>' + title + '</h3><' + tag + '>' + value + '</' + tag + '></li>';
     return '';
 }
 
+function renderLines(title, values, name, tag) {
+    if (tag == undefined)
+       tag = 'big';
+    var result = "";
+    if (values != undefined && values.length)
+       for (var idx = 0 ; idx < values.length ; idx++)
+           if (values[idx][name] != undefined )
+               result += values[idx][name] + ' ';
+    if (result != "") 
+       result = '<li><h3>' + title + '</h3><' + tag + '>' + result + '</' + tag + '></li>';
+    return result;
+}
+
+function renderLinesURL(title, values, name, url) {
+    var result = "";
+    result = '<li><h3>' + title + '</h3><span style="display: inline-block;">';
+    if (values != undefined && values.length) {
+       for (var idx = 0 ; idx < values.length ; idx++) {
+           if (values[idx][url] != undefined)
+               result += '<a href="' + values[idx][url] + '">' + values[idx][name] + '</a><br>';
+           else
+               result += values[idx][name] + '<br>';
+       }
+    }
+    result += '</span></li>';
+    return result;
+}
+
 function renderLineURL(title, URL, display) {
     if (URL != undefined)
-       return '<li><h3>' + title + '</h3> <a href="' + URL + '" target="_blank">' + display + '</a></li>';
+       return '<li><h3>' + title + '</h3><a href="' + URL + '" target="_blank">' + display + '</a></li>';
     return '';
 }
 
@@ -640,7 +883,8 @@ function renderDetails_iphone(data, marker)
 {
        //return renderDetails(data,marker);
 
-
+    if (!data) 
+       return ""; 
     var details = '<div class="details" id="det_'+data.recid+'" >'
 /*
     details = '<div id="header" id="det_'+data.recid+'">' 
@@ -650,26 +894,35 @@ function renderDetails_iphone(data, marker)
 */
     if (marker) 
        details += '<h4>'+ marker + '</h4>'; 
-    details += '<ul class="field">';
+    details += '<ul class="field" >';
     if (data["md-title"] != undefined) {
-       details += '<li><h3>Title</h3> <big> ' + data["md-title"];
-        if (data["md-title-remainder"] !== undefined) {
-             details += ' ' + data["md-title-remainder"] + ' ';
+       details += '<li><h3>Title</h3>' + data["md-title"];
+        if (data["md-title-remainder"] != undefined) {
+             details += ' - ' + data["md-title-remainder"] + ' ';
         }
-        if (data["md-title-responsibility"] !== undefined) {
+/*
+        if (data["md-author"] != undefined) {
+             details += '<i>'+ data["md-author"] +'</i>';
+        } else if (data["md-title-responsibility"] != undefined) {
              details += '<i>'+ data["md-title-responsibility"] +'</i>';
         }
-        details += '</big>'
+*/
         details += '</li>'
     }
+    var coverimagetag = imageHelper.getImageTagByRecId(data.recid, "md-isbn", undefined, "M");
     details 
        +=renderLine('Date',    data["md-date"])
        + renderLine('Author',  data["md-author"])
-       + renderLineURL('URL',  data["md-electronic-url"], data["md-electronic-url"])
-       + renderLine('Subject', data["location"][0]["md-subject"]);
-    
-    if (data["location"][0]["@name"] != undefined)
-       details += renderLine('Location', data["location"][0]["@name"] + " (" +data["location"][0]["@id"] + ")");
+//     + renderLineURL('URL',  data["md-electronic-url"], data["md-electronic-url"])
+       + renderLine('Description',     data["md-description"])
+       + renderLines('Subjects', data["location"], "md-subject");
+
+    if (coverimagetag.length>0) {
+       details += renderLine('Cover', coverimagetag);
+    }
+
+    details += renderLinesURL('Location', data["location"], "@name", "md-url_recipe");
+    details += '<li><a href="#" onclick="showhide(\'recordview\')">Back</a></li>';
     details += '</ul></div>';
     return details;
 }