Any thoughts?

Cheers

Hi Stephen, it seems fine although I cannot test all city from my location. Just as a back up, I would like to add one more line, in case the module for the city doesn't return.

I would like to put an older module that can be relevent for all users. I thought of 2 ways to do it and tested it, but it doesn't work:

//Using else statements:

$mainframe =& JFactory::getApplication();
$template =& $mainframe->getTemplate();

if ( $template == "femmes_jan2011"
and $geoip->city == "Charlesbourg"){ return "450";
}
else {
  return "447";
}


if ( $template == "femmes_jan2011"
and $geoip->city == "Quebec"){ return "458";
}
else {
  return "447";
}


if ( $template == "femmes_jan2011"
and $geoip->city == "Beauport"){ return "459";
}
else {
  return "447";
}


if ( $template == "femmes_jan2011"
and $geoip->city == "Levis"){ return "460";
}
else {
  return "447";
}


if ( $template == "femmes_jan2011"
and $geoip->city == "Montreal"){ return "461";
}
else {
  return "447";
}


if ( $template == "femmes_jan2011"
and $geoip->city == "Neufchatel"){ return "462";
}
else {
  return "447";
}


if ( $template == "femmes_jan2011"
and $geoip->city == "Sainte-foy"){ return "463";
}
else {
  return "447";
}


if ( $template == "femmes_jan2011"
and $geoip->city == "Sillery"){ return "464";
}
else {
  return "447";
}


if ( $template == "femmes_jan2011"
and $geoip->city == "Vanier"){ return "465";
}
else {
  return "447";
}

//Or this solution using $genius at the end of all the above without the "else" bracket:

if ( $template == "femmes_jan2011"
  and $content_genius->check( "module_id not = 458,459,460,461,462,463,464,465")
return "447";


Cheers.
Alex

Great it works (of course)! heh
I did a module for each template instead of coding all the template conditions in the same metamod since I wasn't sure if I needed to open another sub bracket for the next "if template = X" statement.

Thanks a bunch.
Alex.

Ah OK, got it! I knew it was possible but I was just not sure how to code it for a second template.

Would it be something like this:

$mainframe =& JFactory::getApplication();



$template =& $mainframe->getTemplate();



$city = $geoip->city;







if ( $template == "femmes_jan2011" ) {



  if ( $city == "Charlesbourg" ) return "450";



  if ( $city == "Quebec" ) return "458";



  if ( $city == "Beauport" ) return "459";



  if ( $city == "Levis" ) return "460";



  if ( $city == "Montreal" ) return "461";



  if ( $city == "Neufchatel" ) return "462";



  if ( $city == "Sainte-foy" ) return "463";



  if ( $city == "Sillery" ) return "464";



  if ( $city == "Vanier" ) return "465";



  return 447;



}



if ( $template == "TemplateXYZ" ) {



  if ( $city == "Charlesbourg" ) return "450";



  if ( $city == "Quebec" ) return "458";



  if ( $city == "Beauport" ) return "459";



  if ( $city == "Levis" ) return "460";



  if ( $city == "Montreal" ) return "461";



  if ( $city == "Neufchatel" ) return "462";



  if ( $city == "Sainte-foy" ) return "463";



  if ( $city == "Sillery" ) return "464";



  if ( $city == "Vanier" ) return "465";



  return 447;



}

Cheers.
Alex

Thank you Stephen, I did correct one spelling issue. I gave a try to the Coutry check at on top of my rule. I am not sure if it is good.

$mainframe =& JFactory::getApplication();

$template =& $mainframe->getTemplate();

$city = $geoip->city;



if ( $template == "electric_standard_jan2011" ) {



  if ( $fromCountryId == "CA" )

  return 447;

  

    if ( $city == "Charlesbourg" ) return "450";



    if ( $city == "Quebec" ) return "458";



    if ( $city == "Beauport" ) return "459";



    if ( $city == "Lévis" ) return "460";



    if ( $city == "Montreal" ) return "461";



    if ( $city == "Neufchatel" ) return "462";



    if ( $city == "Sainte-foy" ) return "463";



    if ( $city == "Sillery" ) return "464";



    if ( $city == "Vanier" ) return "465";



    return 447;



}

Maybe there's one bracket missing after the first return, or maybe the return is too soon?  Please let me know...

Cheers,
Alex