I'm working on a 4.7 port of "news page". I plan to send it later today. Hope some of you could test it. Regards, eric.

CommentFileSizeAuthor
#10 news_page_0.zip6.2 KBrobert castelo
#5 news_page.zip6.2 KBrobert castelo
#4 news_page_1.install1.02 KBec
#3 news_page_0.install810 bytesec
#2 news_page.install678 bytesec
#1 news_page_EC.module13.53 KBec

Comments

ec’s picture

ec commented
Title: Update to 4.7 » Update to 4.7 -> please test
Status: Active » Needs review
StatusFileSize
new news_page_EC.module13.53 KB

Thanks to Jjeff form updater module, I have something that is working on my test install except the RSS page feed. Please test it before I comit a patch. Regards, Eric.

ec’s picture

ec commented
Title: Update to 4.7 -> please test » Update to 4.7 -> the .install file
StatusFileSize
new news_page.install678 bytes
ec’s picture

ec commented
StatusFileSize
new news_page_0.install810 bytes

add an update existing tables to UTF-8.

ec’s picture

ec commented
StatusFileSize
new news_page_1.install1.02 KB

add update to new database schema from earlier version

robert castelo’s picture

robert castelo commented
StatusFileSize
new news_page.zip6.2 KB

Thanks for your work Eric, it saved me a bit of time updating to 4.7.

Attached is an updated version for 4.7 - I'm not adding it to contrib until I've tested it out more thoroughly, which is why I've uploaded it here.

Does anyone have time to do some testing?

I'm particularly interested in the database update process:

Does 'search' column get created on new install?

Does 'search' column get created on update from 4.6 version?

Does this work OK for pgsql?

ec’s picture

ec commented

Hello Robert, thank you. I will try it later today and let tou know. eric.

ec’s picture

ec commented

Hello Robert,
I tried your zip file, on my current installation, will do more on another one I will upgrade from 4.6 to 4.7. Works fine. But currently I have an issue with the RSS at the end of a page. User with id #1 can't access the feeds even after I gave him all the rights. An in the url there are 2 trailings /, is it normal ? (example : http://localhost/470nets/?q=news_page/feed/170// ). regards, eric.

budda’s picture

budda
Primary language English
Location Cheshire
commented

Weird, the .installfile has a PHP error when i downloaded that zip! I'm attempting to upgrade from a 4.6 install to 4.7

The error is:

Parse error: parse error, unexpected T_LNUMBER in /var/www/html/drupal-4.7/modules/news_page/news_page.install on line 13
llizards’s picture

llizards commented

Same here

Parse error: parse error, unexpected T_LNUMBER in /home/virtual/site7/fst/var/www/html/modules/news_page/news_page.install on line 13
robert castelo’s picture

robert castelo commented
StatusFileSize
new news_page_0.zip6.2 KB

OK, this should do it.....

robert castelo’s picture

robert castelo commented
Title: Update to 4.7 -> the .install file » Update to 4.7 - Help Test
Assigned: ec » robert castelo

...updating title.

robert castelo’s picture

robert castelo commented

New version posted to cvs which is for Drupal 4.7.

There was a mistake in the install file for the version posted on this page.

If you're using that version then disable the module on the admin >> modules page.

Install the new version of the module files.

Check your database, look at table 'system', and delete the row which has the filename column set to the path to the legal module.

Then enable the module again.

Go to your-domain/update.php

Run the update for news_page module.

Lastly, remember that cvs is for development code, so don't use this on a live site unless you've tested it thoroughly on a dummy site.

I'll wait a week for feedback on the cvs version, and if all goes well do a full 4.7 release in a week.

netbjarne’s picture

netbjarne commented

I installed the CVS version, and it did wonders for me for filtering RSS feets. A few notes on usability though:

1) It seems, that you only need to enter the beginning of words to filter by. Ie, one won't have to enter "Japan,Japans,Japaneese" - buts its enough to enter just "Japan". Inline help could be more specific about that.

2) It seems, that when entering words to filter by, you must NOT use spaces in the word list "Red, Green, Blue" - as the module will actually include the leading space in the words (" Green", and " Blue").. The correct way is "Red,Green,Blue". Or so its seems, I'm not sure. A note on that in the inline help, or a strip of leading spaces would be nice..

3) A tip on aggreagating ones own feed created by news page coulkd be mentioned. By doing that, its possible to create a news block with filtered RSS items.

Nevertheless - I think that a 4.7 release of this module would be really nice. I'm using the CVS in production, and I'm very happy with it as it is very very useful - I think the module deserves to be released properly..

robert castelo’s picture

robert castelo commented
Version: master » 4.7.x-1.0
Status: Needs review » Fixed

Official 4.7.x-1.0 release is out.

Thanks for all the feedback and code.

robert castelo’s picture

robert castelo commented
Status: Fixed » Closed (fixed)