HTTPS Problem

[greniers]

I have a problem with the XML map.

All the menu link are "https" instead of "http". the params are sets to "secure":1 ON or "secure":-1 OFF gives always https links.

"secure":0 give an "http" this parameter is for IGNORE.

if I change the file displayer.php
from:
line 155: $node->secure = $item->params->get('secure');
to:
line 155: $node->secure = 0;
the "https" change to "http"

But I need to use both SSL and non-SSL

Edited: SOLUTION?
Finally I remove the changes of the previous file "displayer.php" and change this one

default_class.php into com_xmap\views\xml\tmpl

from:
line 64: $link = JRoute::_($node->link, true, (@$node->secure? 1: -1));

to:
line 64: $link = JRoute::_($node->link, true, ((@$node->secure==0)? -1: $node->secure));

Seems to correct the problem.

versions used are joomla 1.7.3 and xmap 2.2

Thanks for your help

[greniers]

I found that the HTTPS links are not implemented into HTML page

I modified the file com_xmap\views\html\tmpl\default_class.php

from:
line 82: $link = JRoute::_($node->link);

to
line 82: $link = JRoute::_($node->link, true, ((@$node->secure==0)? -1: $node->secure));

It seem to work OK

[greniers]

The 2 lines change above should be:

$link = JRoute::_($node->link, true, @$node->secure==0? -1: $node->secure);

just removed two set of () that not needed.

[greniers]

I'm suprised that theses corrections are not into the latest version 2.2.1

[Richard67]

I'm suprised that theses corrections are not into the latest version 2.2.1

And I am disappointed that these corrections have not been included into meanwhile latest 2.3.0

[guilleva]

Hi, I'm sorry I haven't included this on the latests versions. Can anybody create a pull request for that on Github?

https://github.com/guilleva/Xmap

Please create a fork and then send the pull request.

Thanks,

Guillermo

[Richard67]

Can anybody create a pull request for that on Github?

https://github.com/guilleva/Xmap

Please create a fork and then send the pull request.

Hmm, I am a software developer, but in an old-fashioned company, not open source, so I am new with this github. I've looked into the githup help site to find out what this pull request means but I still am not sure if I understand correctly the roles: Is it normally a job of a developer to fork and so on, or is it just a way to let the developer know it needs some change? What is it good for?

[guilleva]

Hi, fork and pull requests are a very common way to contribute to open source projects. It allows to keep track of those changes and even to give credit to the person who committed the change. You can read more about that on this post:
https://help.github.com/articles/using-pull-requests

[Richard67]

Hi, fork and pull requests are a very common way to contribute to open source projects. It allows to keep track of those changes and even to give credit to the person who committed the change. You can read more about that on this post:
https://help.github.com/articles/using-pull-requests

I've checked the post, and it seems to me now that you want someone here to do the developer job.
I could do so, but it requires a github account, which I haven't.
So who else can do it here?

[guilleva]

Hi,

it looks to me that there is more than one developer here, I'm just saying that the best way to share the changes/fixes is using github and don't get surprised or disappointed if the changes that you shared (which I don't fully understand) were not included on the latest version of Xmap.

I really sorry that I haven't included them, I need to take some time to try to reproduce the problem, that isn't very clear on the original post either, and then apply the fix and test. I haven't had the time yet and I will certainly do it at some point if no one can help with the pull request, or at least, share the modified file (hopefully the file taken from the latest version of Xmap).

Doing pull requests is not doing somebody else's job it's just a much nicer and cleaner way to do what is being done here.

Regards,

Guille