? Success

User tests: Successful: Unsuccessful:

avatar chrisdavenport
chrisdavenport
28 Dec 2016

Pull Request for Issue #13386 .

Summary of Changes

Move web links statistics shown on the admin control panel from Joomla core to a system plugin in the web links package.

Testing Instructions

  • Create at least one published web link.
  • You should now not have any web links entries shown on the admin control panel. Unless this PR is merged and installed first joomla-extensions/weblinks#288.

Documentation Changes Required

None?

avatar chrisdavenport chrisdavenport - open - 28 Dec 2016
avatar chrisdavenport chrisdavenport - change - 28 Dec 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Dec 2016
Category Modules Administration
avatar alikon
alikon - comment - 29 Dec 2016

I have tested this item successfully on 6a767e3


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

avatar alikon alikon - test_item - 29 Dec 2016 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 29 Dec 2016

I have tested this item ? unsuccessfully on 6a767e3

- Apply 13389 (Joomla! CMS): No Weblinks are shown in [mod_admin_stats]

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 29 Dec 2016 - Tested unsuccessfully
avatar chrisdavenport
chrisdavenport - comment - 29 Dec 2016

@franz-wohlkoenig Looks like you might have tried to apply #288 using the patch tester? That won't work because the PR includes a change to the update server manifest to cater for the addition of a new system plugin, but that file is actually not part of the extension package. You need to build the extension package by following the instructions in the README.md and install it using the Joomla extension installer. If you have trouble building the package, drop me a line and I'll send you a pre-built package with PR 288 included.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Dec 2016

@chrisdavenport You are right, i had tried apply #288 by changing Repository of Patchtester. Thanks for Information about building a extension package by my own – i will try it. If i have no success, i write for you're pre-built package.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Dec 2016

@chrisdavenport Can you please send the pre-built package?

avatar yvesh
yvesh - comment - 30 Dec 2016
avatar yvesh
yvesh - comment - 30 Dec 2016

I have tested this item successfully on 6a767e3

Works as expected

screenshot 2016-12-30 09 17 51


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13389.
avatar yvesh yvesh - test_item - 30 Dec 2016 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Dec 2016

I have tested this item successfully on 6a767e3


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13389.
@chrisdavenport @yvesh – thanks for helping

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 30 Dec 2016 - Tested successfully
avatar jeckodevelopment jeckodevelopment - change - 30 Dec 2016
Milestone Added:
Status Pending Ready to Commit
avatar jeckodevelopment jeckodevelopment - change - 30 Dec 2016
Milestone Added:
avatar jeckodevelopment
jeckodevelopment - comment - 30 Dec 2016

RTC


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

avatar rdeutz rdeutz - change - 2 Jan 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-01-02 00:59:04
Closed_By rdeutz
avatar rdeutz rdeutz - close - 2 Jan 2017
avatar rdeutz rdeutz - merge - 2 Jan 2017

Add a Comment

Login with GitHub to post a comment