You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Taher Alkhateeb <sl...@gmail.com> on 2017/09/01 09:17:06 UTC

Re: svn commit: r1806907 - /ofbiz/site/.htaccess

Hi Deepak ...

I don't think you need a rewrite rule. You just need to set the default
links inside the templates

On Sep 1, 2017 12:14 PM, <de...@apache.org> wrote:

> Author: deepak
> Date: Fri Sep  1 09:14:12 2017
> New Revision: 1806907
>
> URL: http://svn.apache.org/viewvc?rev=1806907&view=rev
> Log:
> Added missing redirect rule in htaccess file
>
> Modified:
>     ofbiz/site/.htaccess
>
> Modified: ofbiz/site/.htaccess
> URL: http://svn.apache.org/viewvc/ofbiz/site/.htaccess?rev=
> 1806907&r1=1806906&r2=1806907&view=diff
> ============================================================
> ==================
> --- ofbiz/site/.htaccess (original)
> +++ ofbiz/site/.htaccess Fri Sep  1 09:14:12 2017
> @@ -1 +1,4 @@
> -ErrorDocument 404 /404.html
> +ErrorDocument 404 /404.html
> +
> +RewriteRule ^documentation.html /index.html [R=301,NC,L]
> +RewriteRule ^community.html /getting-involved.html [R=301,NC,L]
>
>
>

Re: svn commit: r1806907 - /ofbiz/site/.htaccess

Posted by Michael Brohl <mi...@ecomify.de>.
Yes, we should definetely tell Google about the permanent redirects using proper 301 rewrites.

Regards,
Michael



Am 02.09.2017 um 14:00 schrieb Ashish Vijaywargiya <as...@hotwaxsystems.com>:

>>> You just need to set the default
> links inside the templates
> 
> Can you please give more details on this? May be some example reference
> will be of great help.
> Deepak discussed the idea with me of writing redirect rules. If any of the
> old URL is crawled in the Google and we don't want to support that URL in
> the new website then we concluded to do a permanent redirect to a specific
> new URL.
> 
> --
> Kind Regards
> Ashish Vijaywargiya
> HotWax Systems - est. 1997 <http://www.hotwaxsystems.com/>
> 
> 
> On Fri, Sep 1, 2017 at 2:47 PM, Taher Alkhateeb <sl...@gmail.com>
> wrote:
> 
>> Hi Deepak ...
>> 
>> I don't think you need a rewrite rule. You just need to set the default
>> links inside the templates
>> 
>>> On Sep 1, 2017 12:14 PM, <de...@apache.org> wrote:
>>> 
>>> Author: deepak
>>> Date: Fri Sep  1 09:14:12 2017
>>> New Revision: 1806907
>>> 
>>> URL: http://svn.apache.org/viewvc?rev=1806907&view=rev
>>> Log:
>>> Added missing redirect rule in htaccess file
>>> 
>>> Modified:
>>>    ofbiz/site/.htaccess
>>> 
>>> Modified: ofbiz/site/.htaccess
>>> URL: http://svn.apache.org/viewvc/ofbiz/site/.htaccess?rev=
>>> 1806907&r1=1806906&r2=1806907&view=diff
>>> ============================================================
>>> ==================
>>> --- ofbiz/site/.htaccess (original)
>>> +++ ofbiz/site/.htaccess Fri Sep  1 09:14:12 2017
>>> @@ -1 +1,4 @@
>>> -ErrorDocument 404 /404.html
>>> +ErrorDocument 404 /404.html
>>> +
>>> +RewriteRule ^documentation.html /index.html [R=301,NC,L]
>>> +RewriteRule ^community.html /getting-involved.html [R=301,NC,L]
>>> 
>>> 
>>> 
>> 

Re: svn commit: r1806907 - /ofbiz/site/.htaccess

Posted by Taher Alkhateeb <sl...@gmail.com>.
Ahhhh right Deepak, my bad, I need to eat more vegetables :) Thank you for
the clarification.

On Sep 4, 2017 8:58 AM, "Deepak Dixit" <de...@hotwaxsystems.com>
wrote:

Hi Taher,

I think you are saying different thing, Here redirect rule is for old url
those are not exists in current site.

What you are saying is to make first link as default link of menu. I and
Ashish discussed the same thing.
I think you are saying in reference of
https://issues.apache.org/jira/browse/OFBIZ-9662





Thanks & Regards
--
Deepak Dixit
www.hotwaxsystems.com
www.hotwax.co

On Sat, Sep 2, 2017 at 11:33 PM, Taher Alkhateeb <slidingfilaments@gmail.com
> wrote:

> Hi Ashish,
>
> I could be wrong, but essentially, the main menu links are left blank
> So instead of redirect, you can just the default inside the link.
>
> So for simple example .. check the below:
>
> Index: template/region/header.tpl.php
> ===================================================================
> --- template/region/header.tpl.php      (revision 1806680)
> +++ template/region/header.tpl.php      (working copy)
> @@ -18,7 +18,7 @@
>                  <li><a href="business-users.html"
> class="last">Business Users</a></li>
>                </ul>
>              </li>
> -            <li><a href="" class="firstLevel<?php if
> (basename($page_name) == 'news.tpl.php'):?> active<?php
> endif;?>">News</a>
> +            <li><a href="//twitter.com/apacheofbiz"
> class="firstLevel<?php if (basename($page_name) == 'news.tpl.php'):?>
> active<?php endif;?>">News</a>
>                <ul>
>                  <li><a href="//twitter.com/apacheofbiz"
> target="external">Twitter</a></li>
>                  <li><a href="//blogs.apache.org/ofbiz/"
> target="external" class="last">Blog</a></li>
>
> On Sat, Sep 2, 2017 at 3:00 PM, Ashish Vijaywargiya
> <as...@hotwaxsystems.com> wrote:
> >>> You just need to set the default
> > links inside the templates
> >
> > Can you please give more details on this? May be some example reference
> > will be of great help.
> > Deepak discussed the idea with me of writing redirect rules. If any of
> the
> > old URL is crawled in the Google and we don't want to support that URL
in
> > the new website then we concluded to do a permanent redirect to a
> specific
> > new URL.
> >
> > --
> > Kind Regards
> > Ashish Vijaywargiya
> > HotWax Systems - est. 1997 <http://www.hotwaxsystems.com/>
> >
> >
> > On Fri, Sep 1, 2017 at 2:47 PM, Taher Alkhateeb <
> slidingfilaments@gmail.com>
> > wrote:
> >
> >> Hi Deepak ...
> >>
> >> I don't think you need a rewrite rule. You just need to set the default
> >> links inside the templates
> >>
> >> On Sep 1, 2017 12:14 PM, <de...@apache.org> wrote:
> >>
> >> > Author: deepak
> >> > Date: Fri Sep  1 09:14:12 2017
> >> > New Revision: 1806907
> >> >
> >> > URL: http://svn.apache.org/viewvc?rev=1806907&view=rev
> >> > Log:
> >> > Added missing redirect rule in htaccess file
> >> >
> >> > Modified:
> >> >     ofbiz/site/.htaccess
> >> >
> >> > Modified: ofbiz/site/.htaccess
> >> > URL: http://svn.apache.org/viewvc/ofbiz/site/.htaccess?rev=
> >> > 1806907&r1=1806906&r2=1806907&view=diff
> >> > ============================================================
> >> > ==================
> >> > --- ofbiz/site/.htaccess (original)
> >> > +++ ofbiz/site/.htaccess Fri Sep  1 09:14:12 2017
> >> > @@ -1 +1,4 @@
> >> > -ErrorDocument 404 /404.html
> >> > +ErrorDocument 404 /404.html
> >> > +
> >> > +RewriteRule ^documentation.html /index.html [R=301,NC,L]
> >> > +RewriteRule ^community.html /getting-involved.html [R=301,NC,L]
> >> >
> >> >
> >> >
> >>
>

Re: svn commit: r1806907 - /ofbiz/site/.htaccess

Posted by Deepak Dixit <de...@hotwaxsystems.com>.
Hi Taher,

I think you are saying different thing, Here redirect rule is for old url
those are not exists in current site.

What you are saying is to make first link as default link of menu. I and
Ashish discussed the same thing.
I think you are saying in reference of
https://issues.apache.org/jira/browse/OFBIZ-9662





Thanks & Regards
--
Deepak Dixit
www.hotwaxsystems.com
www.hotwax.co

On Sat, Sep 2, 2017 at 11:33 PM, Taher Alkhateeb <slidingfilaments@gmail.com
> wrote:

> Hi Ashish,
>
> I could be wrong, but essentially, the main menu links are left blank
> So instead of redirect, you can just the default inside the link.
>
> So for simple example .. check the below:
>
> Index: template/region/header.tpl.php
> ===================================================================
> --- template/region/header.tpl.php      (revision 1806680)
> +++ template/region/header.tpl.php      (working copy)
> @@ -18,7 +18,7 @@
>                  <li><a href="business-users.html"
> class="last">Business Users</a></li>
>                </ul>
>              </li>
> -            <li><a href="" class="firstLevel<?php if
> (basename($page_name) == 'news.tpl.php'):?> active<?php
> endif;?>">News</a>
> +            <li><a href="//twitter.com/apacheofbiz"
> class="firstLevel<?php if (basename($page_name) == 'news.tpl.php'):?>
> active<?php endif;?>">News</a>
>                <ul>
>                  <li><a href="//twitter.com/apacheofbiz"
> target="external">Twitter</a></li>
>                  <li><a href="//blogs.apache.org/ofbiz/"
> target="external" class="last">Blog</a></li>
>
> On Sat, Sep 2, 2017 at 3:00 PM, Ashish Vijaywargiya
> <as...@hotwaxsystems.com> wrote:
> >>> You just need to set the default
> > links inside the templates
> >
> > Can you please give more details on this? May be some example reference
> > will be of great help.
> > Deepak discussed the idea with me of writing redirect rules. If any of
> the
> > old URL is crawled in the Google and we don't want to support that URL in
> > the new website then we concluded to do a permanent redirect to a
> specific
> > new URL.
> >
> > --
> > Kind Regards
> > Ashish Vijaywargiya
> > HotWax Systems - est. 1997 <http://www.hotwaxsystems.com/>
> >
> >
> > On Fri, Sep 1, 2017 at 2:47 PM, Taher Alkhateeb <
> slidingfilaments@gmail.com>
> > wrote:
> >
> >> Hi Deepak ...
> >>
> >> I don't think you need a rewrite rule. You just need to set the default
> >> links inside the templates
> >>
> >> On Sep 1, 2017 12:14 PM, <de...@apache.org> wrote:
> >>
> >> > Author: deepak
> >> > Date: Fri Sep  1 09:14:12 2017
> >> > New Revision: 1806907
> >> >
> >> > URL: http://svn.apache.org/viewvc?rev=1806907&view=rev
> >> > Log:
> >> > Added missing redirect rule in htaccess file
> >> >
> >> > Modified:
> >> >     ofbiz/site/.htaccess
> >> >
> >> > Modified: ofbiz/site/.htaccess
> >> > URL: http://svn.apache.org/viewvc/ofbiz/site/.htaccess?rev=
> >> > 1806907&r1=1806906&r2=1806907&view=diff
> >> > ============================================================
> >> > ==================
> >> > --- ofbiz/site/.htaccess (original)
> >> > +++ ofbiz/site/.htaccess Fri Sep  1 09:14:12 2017
> >> > @@ -1 +1,4 @@
> >> > -ErrorDocument 404 /404.html
> >> > +ErrorDocument 404 /404.html
> >> > +
> >> > +RewriteRule ^documentation.html /index.html [R=301,NC,L]
> >> > +RewriteRule ^community.html /getting-involved.html [R=301,NC,L]
> >> >
> >> >
> >> >
> >>
>

Re: svn commit: r1806907 - /ofbiz/site/.htaccess

Posted by Taher Alkhateeb <sl...@gmail.com>.
Hi Ashish,

I could be wrong, but essentially, the main menu links are left blank
So instead of redirect, you can just the default inside the link.

So for simple example .. check the below:

Index: template/region/header.tpl.php
===================================================================
--- template/region/header.tpl.php      (revision 1806680)
+++ template/region/header.tpl.php      (working copy)
@@ -18,7 +18,7 @@
                 <li><a href="business-users.html"
class="last">Business Users</a></li>
               </ul>
             </li>
-            <li><a href="" class="firstLevel<?php if
(basename($page_name) == 'news.tpl.php'):?> active<?php
endif;?>">News</a>
+            <li><a href="//twitter.com/apacheofbiz"
class="firstLevel<?php if (basename($page_name) == 'news.tpl.php'):?>
active<?php endif;?>">News</a>
               <ul>
                 <li><a href="//twitter.com/apacheofbiz"
target="external">Twitter</a></li>
                 <li><a href="//blogs.apache.org/ofbiz/"
target="external" class="last">Blog</a></li>

On Sat, Sep 2, 2017 at 3:00 PM, Ashish Vijaywargiya
<as...@hotwaxsystems.com> wrote:
>>> You just need to set the default
> links inside the templates
>
> Can you please give more details on this? May be some example reference
> will be of great help.
> Deepak discussed the idea with me of writing redirect rules. If any of the
> old URL is crawled in the Google and we don't want to support that URL in
> the new website then we concluded to do a permanent redirect to a specific
> new URL.
>
> --
> Kind Regards
> Ashish Vijaywargiya
> HotWax Systems - est. 1997 <http://www.hotwaxsystems.com/>
>
>
> On Fri, Sep 1, 2017 at 2:47 PM, Taher Alkhateeb <sl...@gmail.com>
> wrote:
>
>> Hi Deepak ...
>>
>> I don't think you need a rewrite rule. You just need to set the default
>> links inside the templates
>>
>> On Sep 1, 2017 12:14 PM, <de...@apache.org> wrote:
>>
>> > Author: deepak
>> > Date: Fri Sep  1 09:14:12 2017
>> > New Revision: 1806907
>> >
>> > URL: http://svn.apache.org/viewvc?rev=1806907&view=rev
>> > Log:
>> > Added missing redirect rule in htaccess file
>> >
>> > Modified:
>> >     ofbiz/site/.htaccess
>> >
>> > Modified: ofbiz/site/.htaccess
>> > URL: http://svn.apache.org/viewvc/ofbiz/site/.htaccess?rev=
>> > 1806907&r1=1806906&r2=1806907&view=diff
>> > ============================================================
>> > ==================
>> > --- ofbiz/site/.htaccess (original)
>> > +++ ofbiz/site/.htaccess Fri Sep  1 09:14:12 2017
>> > @@ -1 +1,4 @@
>> > -ErrorDocument 404 /404.html
>> > +ErrorDocument 404 /404.html
>> > +
>> > +RewriteRule ^documentation.html /index.html [R=301,NC,L]
>> > +RewriteRule ^community.html /getting-involved.html [R=301,NC,L]
>> >
>> >
>> >
>>

Re: svn commit: r1806907 - /ofbiz/site/.htaccess

Posted by Ashish Vijaywargiya <as...@hotwaxsystems.com>.
>> You just need to set the default
links inside the templates

Can you please give more details on this? May be some example reference
will be of great help.
Deepak discussed the idea with me of writing redirect rules. If any of the
old URL is crawled in the Google and we don't want to support that URL in
the new website then we concluded to do a permanent redirect to a specific
new URL.

--
Kind Regards
Ashish Vijaywargiya
HotWax Systems - est. 1997 <http://www.hotwaxsystems.com/>


On Fri, Sep 1, 2017 at 2:47 PM, Taher Alkhateeb <sl...@gmail.com>
wrote:

> Hi Deepak ...
>
> I don't think you need a rewrite rule. You just need to set the default
> links inside the templates
>
> On Sep 1, 2017 12:14 PM, <de...@apache.org> wrote:
>
> > Author: deepak
> > Date: Fri Sep  1 09:14:12 2017
> > New Revision: 1806907
> >
> > URL: http://svn.apache.org/viewvc?rev=1806907&view=rev
> > Log:
> > Added missing redirect rule in htaccess file
> >
> > Modified:
> >     ofbiz/site/.htaccess
> >
> > Modified: ofbiz/site/.htaccess
> > URL: http://svn.apache.org/viewvc/ofbiz/site/.htaccess?rev=
> > 1806907&r1=1806906&r2=1806907&view=diff
> > ============================================================
> > ==================
> > --- ofbiz/site/.htaccess (original)
> > +++ ofbiz/site/.htaccess Fri Sep  1 09:14:12 2017
> > @@ -1 +1,4 @@
> > -ErrorDocument 404 /404.html
> > +ErrorDocument 404 /404.html
> > +
> > +RewriteRule ^documentation.html /index.html [R=301,NC,L]
> > +RewriteRule ^community.html /getting-involved.html [R=301,NC,L]
> >
> >
> >
>