Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
1 Apr 2015

This PR try to fix this issue: #25

ready to install package: pkg_weblinks341

how to test

install 2.5.28
update to 3.4.1
=> weblinks is broken
install this package: pkg_weblinks341
=> update and weblinks works

avatar zero-24 zero-24 - open - 1 Apr 2015
avatar zero-24
zero-24 - comment - 2 Apr 2015

@mbabker i hope you can have a look here. On testing i notice that this script.php never gets executed.

I have just try to add a coding issue that should throw an error e.g. missing ; or missing <?php but all times i get Installing component was successful. i don't know how to find the issue and fix it? Sorry.

avatar mbabker
mbabker - comment - 2 Apr 2015

From the way I have one of my components structured, it looks like the script.php file is in the wrong place. Try moving it to src/com_weblinks/script.php and building fresh packages.

avatar zero-24
zero-24 - comment - 3 Apr 2015

Thanks @mbabker ! It fails now as expected. Now i will try to implement $db->getTableColumns()

avatar zero-24
zero-24 - comment - 3 Apr 2015

I hope i get is well now. I have update the download package now: pkg_weblinks341

how to test

  • make sure you can install the package successful without errors. (by using the link via install from ULR or download to use and upload and install)
  • do a code review on github.
avatar bembelimen
bembelimen - comment - 3 Apr 2015

@zero-24 I sent you a PR for this PR, see zero-24#1

avatar zero-24
zero-24 - comment - 3 Apr 2015

Thanks @bembelimen just merged :+1:

avatar zero-24
zero-24 - comment - 8 Apr 2015

Now this should also fix #31 Thanks @Re:Later (on the german forum: http://forum.joomla.de/index.php/Thread/256-Fehler-im-Backend-Komponente-Weblinks/?postID=1815#post1815)

The package is updated now and should bring weblinks back to work after upgrading from 2.5.28 to 3.4.1. pkg_weblinks341

Please report back your findings and test results.

avatar hmns
hmns - comment - 8 Apr 2015

after updating from 2.5.28 to 3.4.1 the pkg_weblinks341 was the solution to fix the problems in my weblinks. i installed it with url and in 5 sec all things work as expected.

have fun
Hermann

avatar Sulpher
Sulpher - comment - 8 Apr 2015

It works now! Thanks to @mbabker! :-)

avatar zero-24 zero-24 - reference | - 8 Apr 15
avatar zero-24
zero-24 - comment - 8 Apr 2015

The package is updated again and now match the current state here with the suggestions by @mbabker and @Bakual

The package should bring weblinks back to work after upgrading from 2.5.28 to 3.4.1. pkg_weblinks341

Please report back your findings and test results.

avatar zero-24
zero-24 - comment - 12 Apr 2015

we have 16 downloads from the last version without any more issue reports with the patch. The only one is fixed here: #33 so i guess we can move it to RTC.

avatar zero-24
zero-24 - comment - 29 Apr 2015

We are now to a total count of downloads of 86 without any more issues reported. :smile:

avatar marcodings
marcodings - comment - 14 May 2015

installed successfully and functional on a 3.4.1 site,
upgraded from 2.5x that was using weblinks

avatar zero-24
zero-24 - comment - 14 May 2015

Thanks for your test @marcodings ! Today we have a total download count of 157 for the final download.

avatar wilsonge wilsonge - close - 18 May 2015
avatar zero-24
zero-24 - comment - 18 May 2015

:+1:

avatar zero-24 zero-24 - head_ref_deleted - 18 May 2015
avatar zero-24 zero-24 - change - 18 May 2015
Milestone Added: 3.4.1
avatar alandarr
alandarr - comment - 24 Jul 2015

I am also having problems with this. I have downloaded pkg_weblinks341.zip and was going to install this through the extension manager after uninstalling the previous version. However, I have 3 different extensions already installed and not sure which one/s to uninstall. This was a site that was upgraded from version 2.x

weblinks-extensions

avatar Bakual
Bakual - comment - 24 Jul 2015

You don't uninstall the old version. You install the update over the existing version. That should take care of everything.

avatar alandarr
alandarr - comment - 24 Jul 2015

Trying to install the update over the current version results in these to error messages:


"Warning
Duplicate column name 'version' SQL=ALTER TABLE nux37_weblinks ADD COLUMN version int(10) unsigned NOT NULL DEFAULT '1';

Package Update: There was an error installing an extension: com_weblinks.zip"


Error
Error installing package

avatar Bakual
Bakual - comment - 24 Jul 2015

Please open a new issue so this can be tackled. This one is a closed one.

Add a Comment

Login with GitHub to post a comment