enhancement Failure

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
6 Apr 2017

Summary of Changes

Only support mod_stats and mod_stats_admin

Testing Instructions

Testing Instructions

  • install 3.7.0 (as this is a new plugin event)
  • Create at least one published web link.
  • apply this pr
  • You also need to apply this too: joomla/joomla-cms#15138
  • discover the new plugin
  • install it
  • Enable the system plugin.
  • You should now have an entry about the weblinks count.

Expected result

You should now have an entry about the weblinks count.

Actual result

No weblinks stats in the frontend.

Documentation Changes Required

None.

avatar zero-24 zero-24 - change - 6 Apr 2017
Milestone Added: Weblinks 3.7.0
avatar zero-24 zero-24 - open - 6 Apr 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 8 Apr 2017

seems not possible to apply joomla/joomla-cms#15138 & this PR at once. Installed first #15138, changed in Options Repository to "Weblinks" and couldn't fetch data cause PR is applied.

avatar alikon
alikon - comment - 8 Apr 2017

can you apply this one manually (few lines)...
...don't know if there is a "cool" way

avatar zero-24
zero-24 - comment - 12 Apr 2017

...don't know if there is a "cool" way

the cool way is called git :P But yes this looks like a limitation of patchtester. Can you manually apply the changes?

avatar alikon
alikon - comment - 12 Apr 2017

too cool for my git knowledge level
?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Apr 2017

will try to append manually. I'm so uncool.

avatar zero-24
zero-24 - comment - 12 Apr 2017

Thanks ?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Apr 2017

@zero-24 in #15138 you wrote:

Add the following method to a system plugin:

in which System Plugin and where to add Method?

avatar zero-24
zero-24 - comment - 12 Apr 2017

in which System Plugin and where to add Method?

This is the part i have done here for weblinks ;) So if you have applyed both your are done :)

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Apr 2017

I have to apply this PR and #15138 – both at same Time don't work. Now i try to append Method manually and don't know in which Plugin the Method i have to apply.

avatar zero-24
zero-24 - comment - 12 Apr 2017

if you have the CMS PR applyed. Than you only need to change the files in this PR manually. As i have changed here the weblinks plugin that handle that. So you can test both on the same time ?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Apr 2017

you mean with "System Plugin" the Weblinks-Plugin in there i put Method – anywhere?

avatar zero-24
zero-24 - comment - 12 Apr 2017

You have installed weblinks. So you can just go to this file: plugins/system/weblinks/weblinks.php and do the changes i have done in this Pull request. https://github.com/joomla-extensions/weblinks/pull/338/files than you don't need to add any new moethod as it is allready there.

The example with the new method was if you don't have a method + plugin for that yet.

If you want to add a method you can do this in any system plugin. But you don't need to do that as weblinks already have that method in the file above ?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Apr 2017

Installed weblinks 3.6.0, created a Weblink. Got no Weblinks-Folder:
bildschirmfoto 2017-04-12 um 14 22 37

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Apr 2017

"Components > com_weblinks > weblinks.php"?

avatar zero-24
zero-24 - comment - 12 Apr 2017

Ah you are correct. this is only in the 3.7 version of weblinks wich is not released yet. I can build the plugin itself when i'm back at home today.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Apr 2017

Fine, so i get new Instructions later.

avatar zero-24
zero-24 - comment - 12 Apr 2017

You can apply the CMS PR plus this plugin. (it already include the changes in this PR here)
weblinks.zip

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 13 Apr 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 13 Apr 2017

I have tested this item successfully on 28d147f
Backend
bildschirmfoto 2017-04-13 um 08 19 00
Frontend
bildschirmfoto 2017-04-13 um 08 19 12


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/weblinks/338.
avatar zero-24
zero-24 - comment - 13 Apr 2017

Thanks ?

avatar yvesh yvesh - change - 21 Apr 2017
Status New Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-04-21 17:24:31
Closed_By yvesh
avatar yvesh yvesh - close - 21 Apr 2017
avatar yvesh yvesh - merge - 21 Apr 2017
avatar yvesh yvesh - reference | c6eac62 - 21 Apr 17
avatar yvesh yvesh - merge - 21 Apr 2017
avatar yvesh yvesh - close - 21 Apr 2017
avatar zero-24 zero-24 - head_ref_deleted - 21 Apr 2017
avatar zero-24
zero-24 - comment - 21 Apr 2017

Thanks !

avatar yvesh
yvesh - comment - 21 Apr 2017

Thank you @zero-24

Add a Comment

Login with GitHub to post a comment