Hi Ferdi,
  I could put in almost all the suggestions except $absolute=  TRUE part.
a) When I looked at drupal7 api for the l(text) part, this part was missing:
   https://api.drupal.org/api/drupal/includes%21common.inc/function/l/7 : 
 l($text, $path, array
$options
 = array())

b)Other people in stack-overflows suggested other ways , like 'html' => true and 'external' = true, but they are not working either. 
Is there an example, which could show me the syntax? I tried the following:
a)$combined_page = l($domain.$pageViews,$domain.$pageViews,array('html' => true));
b)$combined_page = l($domain.$pageViews,$domain.$pageViews,array('external' => true));
c)$combined_page = l($domain.$pageViews,$domain.$pageViews,array(),TRUE);

Thanks,
Priya
From: Ferdinand Alimadhi [notifications@github.com]Sent: Friday, May 16, 2014 9:44 AMTo: openscholar/ossapCc: Sudarshanam, PriyashreeSubject: Re: [ossap] made urls clickable links and added /home to portals that are home pages (#35)


In ossap_stats/ossap_stats.module:
> @@ -188,10 +189,13 @@ function ossap_stats_viewed_pages($output_json = FALSE){
>       //as those are not being considered
>       foreach($results['rows'] as $result) {

again ..
$domain = $result[0];
// etc

—
Reply to this email directly or 
view it on GitHub.