Failure

User tests: Successful: Unsuccessful:

avatar astridx
astridx
4 Nov 2017

Pull Request for Issue #367.

Summary of Changes

I changed the batch view of com_weblinks analogously to the batch view of com_content.

Testing Instructions

  1. Create a few weblinks via com_weblinks (Components | Web Links).
  2. Select all this weblinks and click the toolbar button batch
  3. See that the view has a nice margin.
  4. Please also check that the component is still working correctly. For example: Move weblinks to another category or change the tags.
avatar astridx astridx - open - 4 Nov 2017
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 4 Nov 2017 - Tested unsuccessfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 4 Nov 2017

I have tested this item ? unsuccessfully on 5889e02

Modal without PR:

screenshot-2017-11-4 web links - 3 - administration

Modal with PR:

bildschirmfoto 2017-11-04 um 12 06 34

But at Weblinks-View:

bildschirmfoto 2017-11-04 um 12 06 50

i guess this don't depends on PR, to be sure i marked as unsuccessfully.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/weblinks/375.
avatar yvesh
yvesh - comment - 6 Nov 2017

@astridx can you check the issues? Thank you :-)

avatar astridx
astridx - comment - 6 Nov 2017

@franz-wohlkoenig Thank you for testing so reliable.
I tried to adjust your error message. For me, the view is without error. It should look like this:

web links joomla cms test administration

Or the column associations should be empty, if there is no association.

My first idea:
Line 152 of the file JOOMLA/administrator/components/com_weblinks/views/weblinks/tmpl/default.php need the variable associations. This variable was introduced in this PR a few days ago:
#349

Could it be, that this PR is not merged well in your testing installation? How did you update your system?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 7 Nov 2017

I use pkg-weblinks-3.7.0-beta1.zip. I can download it once more an test again if you say so.

Using Branch "Master" or "Test": Install failed.

avatar astridx
astridx - comment - 7 Nov 2017

@franz-wohlkoenig I'm not sure, but I think pkg-weblinks-3.7.0-beta1.zip only includes changes made before May (https://github.com/joomla-extensions/weblinks/releases).

@ALL Are I am right? Does any one know if there is a nighly build for web links?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 7 Nov 2017

I think so too but didn't find a actual Source.

avatar astridx
astridx - comment - 7 Nov 2017

@franz-wohlkoenig If you like you can try with the source I uploaded here a few minutes ago: #377

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 8 Nov 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 8 Nov 2017

I have tested this item successfully on 5889e02


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/weblinks/375.

avatar astridx
astridx - comment - 8 Nov 2017

@franz-wohlkoenig Thank you very much for testing.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 9 Nov 2017

thanks for your Work @astridx.

avatar alikon
alikon - comment - 9 Feb 2018

I have tested this item successfully on 5889e02


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/weblinks/375.

avatar alikon alikon - test_item - 9 Feb 2018 - Tested successfully
avatar astridx
astridx - comment - 10 Feb 2018

Thanks for the test @alikon
It would be great if some could merge take. There are now three successful tests.

Can we create a new Tag?

avatar yvesh yvesh - close - 12 Feb 2018
avatar yvesh yvesh - change - 12 Feb 2018
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-02-12 22:58:05
Closed_By yvesh
avatar yvesh yvesh - close - 12 Feb 2018
avatar yvesh yvesh - merge - 12 Feb 2018
avatar yvesh yvesh - reference | 8574a4f - 12 Feb 18
avatar yvesh yvesh - merge - 12 Feb 2018
avatar yvesh
yvesh - comment - 12 Feb 2018

Thanks @astridx .. Merged it in the master branch now.. we need to bump version anyways and do another beta / rc to get all the improvements of the last months out.

Add a Comment

Login with GitHub to post a comment