Welcome to the MetaMod and Chameleon Support Forums.

Before posting, please check out the FAQs.

helpme

 

Need extra help with your Joomla site? Consider paid Joomla support by the developer of Chameleon and MetaMod.

 

Page: 1

Condense code: else if's

The Permalink allows you to share this Post with Friends and other Users. To use, click the Permalink and then copy the URL from your Browsers Adress Bar.#1  Permalink    Top 20-Jun-11 21:46:35

Condense code: else if's

This IS working but would like to condense it:

if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '1'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '2'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '3'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '4'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '5'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '6'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '7'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '8'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '9'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;
else;
if (
$option == 'com_hotproperty'
and $view == 'types'
and $id == '10'
and $Itemid == '65'
and JRequest::getVar('layout') == 'properties'
) return;

else return 113;

I attempted and $id == "1,2,3,4,5,6,7,8,9,10'
didn't work...

Thank you!

zbabyface
Beginner Modder
ranks
useravatar
Offline
5 Posts
User info in posts
Administrator has disabled public posting
The Permalink allows you to share this Post with Friends and other Users. To use, click the Permalink and then copy the URL from your Browsers Adress Bar.#2  Permalink    Top 22-Jun-11 22:48:05

Re: Condense code: else if's

Ok, how about this version:

if (
  $option == 'com_hotproperty'
  and $view == 'types'
  and JRequest::getVar('layout') == 'properties'
) {
  $exclusions = array(1,2,3,4,5,6,7,8,9,10);
  if (in_array( $id, $exclusions )) return;
}
return 113;

Or, even simpler since you are using ids 1 to 10 consecutively:

if (
  $option == 'com_hotproperty'
  and $view == 'types'
  and JRequest::getVar('layout') == 'properties'
  and $id >= 1 and $id <= 10
) return;
return 113;


I deliberately left out the Itemid check, as that is equivalent to checking for the menu item. If you assign the MetaMod to only that menu item, then it's redundant in the PHP. If you assign the MetaMod to ALL menu items, then it might be useful to not check for the Itemid, so that the rule will catch the URLs even if the Itemid gets screwed up.

Hope that helps,
Stephen

Stephen Brandon
MetaMod / Chameleon developer
If you use MetaMod or Chameleon, please post a rating and a review at the Joomla! Extensions Directory: Chameleon | MetaMod

metamodguy
useravatar
Offline
3329 Posts
User info in posts
Administrator has disabled public posting
Page: 1

Board Info

Board Stats:
 
Total Topics:
1684
Total Polls:
6
Total Posts:
5940
Posts this week:
1
User Info:
 
Total Users:
5161
Newest User:
kbmgkey234q
Members Online:
1
Guests Online:
175

Online: 
visability

Forum Legend:

 Topic
 New
 Locked
 Sticky
 Active
 New/Active
 New/Locked
 New Sticky
 Locked/Active
 Active/Sticky
 Sticky/Locked
 Sticky/Active/Locked