[Drache]

I want to make this hack only show for step 2. Just to remind people to register their account. I changed the code like this could you please tell me if i did it right?

From

PHP Code:

<if condition="THIS_SCRIPT != 'register' AND THIS_SCRIPT != 'login'">

<if condition="$show['guest']"> 
    <!-- guest welcome message --> 
    <br />
    <table class="tborder" cellpadding="$stylevar[cellpadding]" cellspacing="$stylevar[cellspacing]" border="0" width="100%" align="center">
    <tr> 
        <td class="alt1">
            <phrase 1="faq.php?$session[sessionurl]" 2="register.php?$session[sessionurl]" 3="$vboptions[bbtitle]">$vbphrase[first_visit_message]</phrase>
        </td>
    </tr>
    </table>
    <!-- / guest welcome message -->
<else /> 

<if condition="is_member_of($bbuserinfo, 3)">
    <!-- / unconfirmed user message -->
    <br />
    <table class="tborder" cellpadding="$stylevar[cellpadding]" cellspacing="$stylevar[cellspacing]" border="0" width="100%" align="center">
    <tr> 
        <td class="alt1">
            <phrase 1="$vboptions[bbtitle]">$vbphrase[welcome_unconfirmed]</phrase>
        </td>
    </tr> 
    </table>
    <!-- / unconfirmed user message -->
<else />

<if condition="$bbuserinfo[posts]<1">
    <!-- never posted message -->
    <br />
    <table class="tborder" cellpadding="$stylevar[cellpadding]" cellspacing="$stylevar[cellspacing]" border="0" width="100%" align="center">
    <tr>
        <td class="alt1">
            <phrase 1="$bbuserinfo[username]" 2="$vboptions[bbtitle]">$vbphrase[welcome_neverposted]</phrase>
        </td>
    </tr>
    </table>  
    <!-- / never posted message -->
<else /> 

<if condition="$headerstime - 1209600 > $bbuserinfo[lastpost]">
    <!-- hasnt posted lately message -->
    <br /> 
    <table class="tborder" cellpadding="$stylevar[cellpadding]" cellspacing="$stylevar[cellspacing]" border="0" width="100%" align="center"> 
    <tr> 
        <td class="alt1">
            <phrase 1="$bbuserinfo[username]" 2="$vboptions[bbtitle]">$vbphrase[welcome_postmoreoften]</phrase>
        </td>
    </tr>
    </table>
    <!-- / hasnt posted lately message -->
</if>
</if>
</if>
</if>
</if> 


To

PHP Code:

<if condition="is_member_of($bbuserinfo, 3)">
    <!-- / unconfirmed user message -->
    <br />
    <table class="tborder" cellpadding="$stylevar[cellpadding]" cellspacing="$stylevar[cellspacing]" border="0" width="100%" align="center">
    <tr> 
        <td class="alt1">
            <phrase 1="$vboptions[bbtitle]">$vbphrase[welcome_unconfirmed]</phrase>
        </td>
    </tr> 
    </table>
    <!-- / unconfirmed user message -->
<else />
</if> 


Edit. I've figured it out and got it working. Awsome hack. Thanks