? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
19 Mar 2017

Pull Request for Issue #13816

Summary of Changes

This just removes the never needed nor used option to switch SSL of per menu option.

Testing Instructions

Confirm that the off option is gone if you edit a menu item

Expected result

off option is gone

Actual result

Off option is there but never implemented.

Documentation Changes Required

None.

avatar zero-24 zero-24 - open - 19 Mar 2017
avatar zero-24 zero-24 - change - 19 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 19 Mar 2017
Category Administration com_menus
avatar brianteeman
brianteeman - comment - 19 Mar 2017

It is used. It describes the state

avatar zero-24
zero-24 - comment - 19 Mar 2017

It is used. It describes the state

I do not understand that? If you set this option to -1 / Off there is no code which checks that nor makes that sense. If you force https this setting is ignored. And only On makes sense here. Atleast for me.

avatar brianteeman
brianteeman - comment - 19 Mar 2017

Don't think about the code. Think about the user interface.

On 19 Mar 2017 12:52 p.m., "zero-24" notifications@github.com wrote:

It is used. It describes the state

I do not understand that? If you set this option to -1 / Off there is no
code which checks that nor makes that sense. If you force https this
setting is ignored. And only On makes sense here. Atleast for me.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#14771 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8XHPp24aI-OwoGY1dqbVLhUCukiYks5rnSTzgaJpZM4MhuRa
.

avatar zero-24
zero-24 - comment - 19 Mar 2017

Don't think about the code. Think about the user interface.

Sure but for what reason you want to swich that off?

On => Overrride the global setting wich says "don't force https" if we force HTTPS this setting is ignored. (it is equal)
ignore => use the global settings.
Off => Never works nor can it. If the global setting is forced off this setting is makes no sense.

btw. even if you enabled https just to one page there is (at least for me) no need to set this option Off ;)

avatar brianteeman
brianteeman - comment - 19 Mar 2017

It is not about what it does. It's about what it tells the user.

avatar zero-24
zero-24 - comment - 19 Mar 2017

It's about what it tells the user.

It tells the user he switched it off but this don't work?

avatar brianteeman
brianteeman - comment - 19 Mar 2017

Show me any other example where you have an On option but do not have an
off.

The solution is not to simply remove the off string.

On 19 Mar 2017 1:19 p.m., "zero-24" notifications@github.com wrote:

It's about what it tells the user.

It tells the user he switched it off but this don't work?


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#14771 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8eJA6F6k_IKRmiN9z5UscoAO-K0Dks5rnStbgaJpZM4MhuRa
.

avatar AlexRed AlexRed - test_item - 19 Mar 2017 - Tested successfully
avatar AlexRed
AlexRed - comment - 19 Mar 2017

I have tested this item successfully on ebce41a

Patch ok for me. Thanks


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14771.

avatar brianteeman brianteeman - test_item - 19 Mar 2017 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 19 Mar 2017

I have tested this item ? unsuccessfully on ebce41a

if you upgrade a site which had been saved with -1 aka off
then if you go to the menu item it will be displayed as being on


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14771.
avatar brianteeman
brianteeman - comment - 19 Mar 2017

screenshotr14-14-35

{"show_title":"1","link_titles":"0","show_intro":"","info_block_position":"","info_block_show_title":"","show_category":"0","link_category":"","show_parent_category":"0","link_parent_category":"","show_associations":"","show_author":"0","link_author":"","show_create_date":"0","show_modify_date":"0","show_publish_date":"0","show_item_navigation":"0","show_vote":"","show_icons":"","show_print_icon":"","show_email_icon":"","show_hits":"0","show_tags":"","show_noauth":"","urls_position":"","menu-anchor_title":"","menu-anchor_css":"","menu_image":"","menu_text":1,"menu_show":1,"page_title":"","show_page_heading":"0","page_heading":"","pageclass_sfx":"","menu-meta_description":"","menu-meta_keywords":"","robots":"","secure":-1}

avatar AlexRed
AlexRed - comment - 19 Mar 2017

yes, I confirm the problem after upgrade. Need an if OFF after upgrade --> Ignore

avatar Twincarb
Twincarb - comment - 19 Mar 2017

I have a site which has 1 page that can't be https all other pages are set to on apart from this single page set to off, this ensures that any visitor arriving from another page on the site see's the page correctly.
leaving the page set to ignore would mean that going from one https page to this one would cause the page to not load correctly.

For there to be logic here, if Force HTTPS is set to Entire site then showing the option in menus should be hidden and ignored, possibly with a non-changeable state of SECURE/ON/ENABLED if the global setting is set to None or Administrator only the menu option is valid and should be selectable.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14771.
avatar AlexRed
AlexRed - comment - 2 May 2017

I can confirm the problem on Joomla! 3.7.1-dev (nightly Build)

avatar brianteeman
brianteeman - comment - 2 May 2017

@alexred as its still open then of course nothing has changed.

avatar rdeutz rdeutz - change - 27 May 2017
Labels Added: ?
avatar zero-24 zero-24 - change - 11 Jun 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-06-11 21:45:52
Closed_By zero-24
avatar zero-24 zero-24 - close - 11 Jun 2017

Add a Comment

Login with GitHub to post a comment