Failure

User tests: Successful: Unsuccessful:

avatar astridx
astridx
5 Mar 2017

Pull Request for Issue #211.

Summary of Changes

Add a basic front end view so that zero-24 can go on with the rendering in the frontend for custom fields #317

Testing Instructions

Apply this patch on a 3.7 version

Expected result

Your can apply a single weblink for a menu item in the frontend.

Actual result

None
I've been looking at com_content.

Documentation Changes Required

Yes

Addition
I just realized, that I forgot to put the weblink in the frontend view into a <a href=>- tag. I will add that if you like my suggestion so far.

I've been picked much from com_content.

avatar astridx astridx - open - 5 Mar 2017
avatar astridx astridx - change - 5 Mar 2017
The description was changed
avatar astridx astridx - edited - 5 Mar 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Mar 2017

After installing PR got Error: The file marked for modification does not exist: administrator/components/com_weblinks/weblinks.xml, PR not installed.

avatar zero-24 zero-24 - change - 5 Mar 2017
Milestone Added: Weblinks 3.7.0
avatar astridx
astridx - comment - 6 Mar 2017

@franz-wohlkoenig Thank you for testing.

I just tried to apply the patch via patch tester. For me it worked.
I used patch tester version 3.0.0 beta (https://github.com/joomla-extensions/patchtester/releases/tag/3.0.0-beta2) on the current Joomla! 3.7.0-rc1 dev.

Perhaps someone else knows why @franz-wohlkoenig gets the error message?

avatar astridx astridx - change - 6 Mar 2017
The description was changed
avatar astridx astridx - edited - 6 Mar 2017
avatar astridx astridx - change - 6 Mar 2017
The description was changed
avatar astridx astridx - edited - 6 Mar 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Mar 2017

i know: to many reinstalls, so no weblinks installed.

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 6 Mar 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Mar 2017

I have tested this item successfully on cc7df89

  • Title and Url is shown, not clickable
  • Weblink created in Modal is locked (similar Categories and Articles)

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/weblinks/325.
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Mar 2017

i have tested against 3.7.0-rc1, as i know should be tested on 3.6.5 but theres now the 3.7-milestone.

avatar zero-24
zero-24 - comment - 6 Mar 2017

@franz-wohlkoenig we are going to release a 3.6 version of weblinks soon ? So anything new goes into the 3.7 version of weblinks.

avatar astridx
astridx - comment - 10 Mar 2017

@franz-wohlkoenig Thank you for testing :)

I just added the forgotton link to the Url. I did not link the title.
If someone else has other ideas and I can implement that, I will do this. I'm not very creative at the moment :( I am also open to any criticism.

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 11 Mar 2017 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Mar 2017

I have tested this item ? unsuccessfully on 5bb713b

Menu Single Weblink: Click on Select Weblink > in Modal click on an Weblink got no response.
No Message on JS in Browser-Console.

Test on

weblinks-3.6.0
macOS Sierra, 10.12.3
Firefox 52.0 (64-bit)

MAMP 4.1.1

  • PHP 7.0.15
  • MySQLi 5.6.35

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/weblinks/325.
avatar astridx
astridx - comment - 11 Mar 2017

Thank you very much for your tests.
If I understand you correctly, your problem occurred when creating the menu item for the single weblink. I did not really change anything in the code for this.
Did you assign the patch with the help of the pathtester component? Did you tested on the same machine as last time?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Mar 2017

Problem is to click on an Weblink in the Modal – noting happens. So you cant select a Weblink to assign to Menu.

I added this PR on same Machine, Difference is i use latest nightly builds from Today.

avatar astridx
astridx - comment - 11 Mar 2017

Thank you for the quick reply :)

avatar astridx astridx - change - 24 Mar 2017
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-03-24 10:51:32
Closed_By astridx
avatar astridx astridx - close - 24 Mar 2017
avatar astridx astridx - merge - 24 Mar 2017
avatar astridx astridx - reference | 3a32a86 - 24 Mar 17
avatar astridx astridx - merge - 24 Mar 2017
avatar astridx astridx - close - 24 Mar 2017
avatar astridx
astridx - comment - 24 Mar 2017

Accidently I merged this. I made a revert (#332) and I will create a new PR.
Sorry for this.

avatar yvesh
yvesh - comment - 24 Mar 2017

@astridx no issue, keep it merged we continue from here.

avatar zero-24 zero-24 - change - 28 Mar 2017
Milestone Removed: Weblinks 3.7.0

Add a Comment

Login with GitHub to post a comment