?
avatar hakanara
hakanara
26 Dec 2015

I followed all the steps of "Editing Restricting access to Articles individually" as stated here:
https://docs.joomla.org/Restricting_access_to_%22read_more%22

As a result, in the blog view, the article titles are not clickable although the opposite is selected in the menu settings. (Administrator -> Menu of the blogged category -> Options -> Linked titles -> Yes)

But when I login, the titles are clickable. I copy the link when I am logged in, I log out and paste the link to the browser, I access to the page where the first half of the text before "read me" is displayed. So actually the link works. So why can not I make it clickable from the menu settings?

Logically, if the link shows the first part of the restricted content (the part before read more) when pasted it into the browser, it should be clickable in the blog view. As far as I remember, it was clickable before.

(On top, I think because of these settings, when I use a sitemap component (like OSmap) the links are not shown on the sitemap.)

For the search engines to follow the first part of the content, I propose the links to be clickable on the blog layout if the opposite is not set in the menu settings.

avatar hakanara hakanara - open - 26 Dec 2015
avatar sovainfo
sovainfo - comment - 26 Dec 2015

Recall that to be a known bug, no solution provided.
Suggest to use globally instead of the menu item (individually).


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

avatar hakanara
hakanara - comment - 26 Dec 2015

Thank you, can you elaborate your suggestion please? What does "use globally" mean? You mean set each article individually?

avatar sovainfo
sovainfo - comment - 26 Dec 2015

No, the document you referred to mentions 1. individually and 2. global.
You mentioned you opted for 1 indivually.
2. global is at the component level.

avatar hakanara
hakanara - comment - 26 Dec 2015

There is no change. The title links are still unclickable. I tried many different combinations. Does it work for you if you do it as the 2nd option?

avatar sovainfo
sovainfo - comment - 26 Dec 2015

Don't know, not using that functionality.

avatar creativeprogramming
creativeprogramming - comment - 30 Dec 2015

Making this change should solve your issue: staging...creativeprogramming:patch-3

I commented the '&& $params->get('access-view')' in layouts/joomla/content/blog_style_default_item_title.php

avatar hakanara
hakanara - comment - 30 Dec 2015

Yes it worked @creativeprogramming Grazie mille. So are you going to add the new parameters now? Joomla team rocks :-)

avatar creativeprogramming
creativeprogramming - comment - 31 Dec 2015

@hakanara no, I'm a Joomla user like you, not part of the Joomla team, I provided just a way for you to fix and some 'suggestion/inspiration' for the team to locate where the issue is, and the 'TODO:' i wrote in the code comments is part of my suggestion: let's hope someone in the team will take this and improve it (I didn't made a PR as I know that my time is not enough to test and use the proper code styling, I just want to help to fix issues, not to merge my code).

Note: there are also other views to fix with the same issue (that can be fixed in the same way) and I think adding a parameter is a good thing, anyway they may also simply remove the '$params->get('access-view')' check as it's unneeded for security as in the item view, like you stated, is instead used just to hide the 'fulltext' but not to prevent access to the URL, so having links to it should not be prevented

avatar hakanara
hakanara - comment - 31 Dec 2015

Actually there is already a parameter for this (Show title with links or something like this) So if the article is blocked by access restrictions, maybe there would be no need for hiding the link of the article. Let the developers discuss it.

avatar hakanara
hakanara - comment - 3 Jan 2016

@creativeprogramming You gave me an idea. I copied the modifed file to the html folder of the template and it works. Thank you. Hopefully, Joomla developers will respond to this :-)

avatar sovainfo
sovainfo - comment - 3 Jan 2016

Unfortunately that is not the way it works here. Without a PR with the suggested code change nothing is going to happen. You might be lucky and find someone that produces the PR. Doubt that very much because both solutions provided by @creativeprogramming are not of the quality to be applied to core.

Advice against using the modified code even as an template override. It is clear that you don't understand the meaning of the setting access-view. Removing that condition indeed gets you the linked title, only for more situations than you want. Obviously you need to restrict it to the situation at hand. Which is when you configured to show the teaser.

So, the condition would be:
if ($params->get('link_titles') && ($params->get('access-view') || $params->get('show_noauth', '0') == '1'))

avatar hakanara
hakanara - comment - 3 Jan 2016

@sovainfo , thank you very much. Your recommendation also worked for me. I managed to solve the situation with the template override. I know that the developers should agree on a solution and the code should be added to the package with a PR. I don't want to create many discussions, but your recommendation in the last message could be implemented to the core, right?

avatar sovainfo sovainfo - reference | 002eae6 - 3 Jan 16
avatar sovainfo
sovainfo - comment - 3 Jan 2016

Created #8827
Please mention your test result using issues.joomla.org on #8827

With thanks to @creativeprogramming for pointing to the troubled code.

Suggest to close this issue, as there is a PR.

avatar brianteeman brianteeman - close - 3 Jan 2016
avatar hakanara
hakanara - comment - 3 Jan 2016

@sovainfo Thank you very much. Great team here.

I have tested all the combinations (authorized links = yes/no, clickable title links = yes/no, and acess rights = registered) and it worked as expected. So I close the issue now.

avatar hakanara hakanara - close - 3 Jan 2016
avatar hakanara hakanara - close - 3 Jan 2016
avatar hakanara hakanara - change - 3 Jan 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-01-03 22:22:09
Closed_By hakanara
avatar brianteeman brianteeman - change - 8 Mar 2016
Labels Added: ?

Add a Comment

Login with GitHub to post a comment