Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update problem. #74

Open
aerograf opened this issue Feb 15, 2019 · 16 comments
Open

Update problem. #74

aerograf opened this issue Feb 15, 2019 · 16 comments
Labels

Comments

@aerograf
Copy link
Member

The first thing I encountered before starting the test is not a valid update, which led to the loss of the previous database (test). Could do something wrong, but then you need to write how to carry out the update.

@aerograf
Copy link
Member Author

Upgrade from 1.66 to 2.0
XOOPS 2.5.10-Beta

@GregMage
Copy link
Collaborator

@aerograf
Copy link
Member Author

Yes, but the DB did not see and did not rename the database tables to the new format.
I'll try again.

@aerograf
Copy link
Member Author

Module File for TDMDownloads Not Found!....
Unfortunately, it does not work. No matter how I did it. Basically writing an error with
Tdmdownloads \ Helper. If you start to change to TDMDownloads \ Helper, then the module stops working.
Installation on a clean, runs flawlessly.

@GregMage
Copy link
Collaborator

The problem: You have already updated from 1.66 to 2.0 (even if there was an error). It is necessary to delete the file tdmdownloads and to put back the files of the version 1.66 and to make an update of the module. The module is now in version 1.66. You follow the next procedure and all is well.

https://github.com/XoopsModules25x/tdmdownloads/blob/master/docs/Update.txt

@aerograf
Copy link
Member Author

I'll try later what you offer.
There was a problem with "render ()". For some reason, display () works, and render () is not ... Because of this, the module in the admin panel does not display pages.

@cesagonchu
Copy link
Collaborator

A solution, while waiting to find better:
https://www.frxoops.org/modules/newbb/viewtopic.php?post_id=242657#forumpost242657

@mambax7
Copy link
Contributor

mambax7 commented Aug 8, 2021

Please ask babouille to fork the module and upload his changes. This will help to keep everything in one place :)

@mambax7
Copy link
Contributor

mambax7 commented Aug 8, 2021

And it's always good to look at the network diagram to see who has done what, and what are the latest versions, so we're aware of any overlaps:
https://github.com/XoopsModules25x/tdmdownloads/network

@cesagonchu
Copy link
Collaborator

Ok, but the only change he made was to rename tdmdownloads and everything else that calls the module to TDMDownloads.
Users who need to update the module can't because the module has been renamed to tdmdownloads (in lower case).

@mambax7
Copy link
Contributor

mambax7 commented Aug 8, 2021

They can update it by renaming the folder in the /modules on their server to "tdmdownloads"
It's for consistency how we use namespaces...

@cesagonchu
Copy link
Collaborator

babouille tried and this method did not work.
Do you have a tutorial?

@mambax7
Copy link
Contributor

mambax7 commented Aug 9, 2021

What version of TdmDownloads was he upgrading from? I could run the test here locally...
And what exactly did go wrong there?

@cesagonchu
Copy link
Collaborator

We, 1.5 from 2010 and a user tried with 1.65

@cesagonchu
Copy link
Collaborator

cesagonchu commented Aug 9, 2021

We got "TDMDownloads not found" and the user got Error: Class 'TDMObjectTree' not found"
Then he got this:
Error: Class 'XoopsModules\Tdmdownloads\Utility' not found "

@mambax7
Copy link
Contributor

mambax7 commented Aug 9, 2021

A lot of the issues are related to the fact that "TDMDownloads" was hard coded in the module, so any change in in the module to "tdmdownloads" without also changing the name of the folder, will break it. I'll look into the issue and how to make the update easier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants