-
Notifications
You must be signed in to change notification settings - Fork 31
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
Articles Extra Fails to Update #159
Comments
Hi @visualgui,
|
@rthrash might there be something wrong with the package at the providers side? It's the second time someone reported this issue. |
@JoshuaLuckers If the package was created correctly and the uploaded transport.zip was not corrupt it should work. We'd have all sorts of complaints if there were something wrong with the Extras repo. We'd need more info from the environment and any errors displayed in the log, php log or other logs per @muzzwood's ask. Also, I'd like to hear more about what this means, @visualgui "Pages created with Articles Extra are completely broken"? |
@jaygilmore @JoshuaLuckers I just tried installing 2.0.0-alpha1 from the MODX repo and got the same error. The POST response is:
I'm guessing that means that pages aren't showing/loading in the tree since it won't be the 3.x compatible version. (wrong class_key) Error log shows:
The above is on a MODX Cloud instance running MODX 3.0.0-pl with PHP 8.0. (also downgraded to PHP 7.4, same thing) I can confirm that uploading the package and installing it that way works on the same instance. @visualgui until this is sorted, you can download the package from here: https://github.com/modxcms/Articles/blob/develop-3x/_packages/articles-2.0.0-alpha1.transport.zip |
@jaygilmore, the pages were not loading at all, and as @muzzwood pointed out, the directory that has the articles disappeared from the tree. @muzzwood, Unfortunately I did not update to 1.8 before I upgrade to MODX 3.0.0. I am running on MODX Cloud instance with PHP 7.3. I tried to upload the package you mentioned and this is the error log: `Console running...
|
Thanks for reporting back @visualgui Did the installation complete, or did it fail? Or just stop there? |
@muzzwood It stopped the first time. I just tried it again and it is now successfully installed. The pages now loaded, but get a 404. The articles in the tree is still missing. |
@visualgui are there any console errors? Is it possible of stuck browser cache? I'm referring to the issue of the missing Resource from the tree. |
@visualgui Delete your cache if you can. Otherwise, I wonder if the dbfields resolver failed for some reason: https://github.com/modxcms/Articles/blob/develop-3x/_build/resolvers/dbfields.resolver.php If you're able to use PHPMyAdmin (or similar) to look at your database, have a look at the |
@jaygilmore, yes there were console errors, but I didn't save it. Is it being saved somewhere else I can grab? @muzzwood, I cleared the cached and it is still not showing up. Unrelated topic: Is it safe to switch back from PHP 8.0 to 7.4 on MODX Cloud? When I switched from 7.4 to 8.0, the entire site went down so I had to switch it back. It looks like everything seem to work fine, but I wonder if I screwed up something. |
@visualgui you can switch back to 7.4. It will not cause any issues. Once you get Articles working, I'd try and figure out what Extras you have installed that are causing the site to fail on 8. PHP 7.4 is supported only until December 2022. PHP 8.1 will be available on the new platforms in Cloud in the coming weeks. Console errors are ephemeral (unless you turn on retention) as they are generated by the browser while the operations are running and until you access a new URI. It is possible to set persistent logging to on in most browser dev tools. |
Thanks @jaygilmore. That’s reassuring. |
I believe everything went fine with uploading the package. The only thing I did notice was that I didn't get an confirmation email about the new version being approved. For the 1.8 release I did get an email. That's why I was thinking maybe it's something in that area. |
Hi @visualgui, |
|
@muzzwood, I cleared the cache and refreshed the tree. Still no articles directory and nothing in the error log. I reinstalled articles extra, here's the console errors: `Console running...
|
@jaygilmore Editing the current listing and re-uploading the Transport Package File solved the "package could not be downloaded" issue. |
@JoshuaLuckers That's really odd. Do you recall any issue other than the lack of email? |
It sure is, maybe I did upload a corrupt file or the wrong one. |
FYI: I had the same problem ( “Could not download and create transport package with signature: articles-2.0.0-alpha1”), but after upgrading MODx to 3.0.1, it was installed. Unrelated to this issue, I have some other ones, but will have to investigate and sort out if it's bugs or me who hasn't configurated everything properly. I'm test running everything on Kubuntu 20.04.4 with Apache 2.4.41, PHP 7.4.3, and MariaDB 10.3.34. |
😮 That I have not seen before! Edit: Actually, I just updated to 3.0.1 and I'm getting the same thing. I think there was some reworking on resource panels in the core and that messed up the articles resources. As a quick fix (until the next release), open up the file:
and comment out line 58 e.g. |
@visualgui
|
@muzzwood, I tried your code, but it didn’t work. I think it might have to do with this template. The following line, in particular, broke the page: I wonder if something changed from 2.8 to 3.0. Here's the live page: https://ho8tajk7.modx.dev/pubs/papers/ |
Do you mean the fix for the container layout, or the SQL query?
Yeah the way placeholders work changed a bit in MODX 3. Try calling the tags uncached with an exclamation mark. e.g. |
Bug report
Summary
Pages created with Articles Extra are completely broken when upgraded to MODX 3.0.0
Step to reproduce
Tried to update Article Extra, but it failed.
Observed behavior
Could not download and create transport package with signature: articles-2.0.0-alpha1
Expected behavior
Here’s the failure message: “Could not download and create transport package with signature: articles-2.0.0-alpha1”
Environment
Articles version 1.7.13-pl
The text was updated successfully, but these errors were encountered: