Please port to D6
Babalu - September 26, 2008 - 18:23
| Project: | Simplenews Scheduler |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | sgabe |
| Status: | needs work |
Description
Please port this module to D6
#1
subscribing
#2
subscribes
#3
subscribing
#4
Yes, please! THANKS!
#5
+1
#6
add me to the requesters
#7
+1
#8
+2
#9
subscribes
#10
Any news?
#11
Anyone working on this?
#12
+1000
#13
iam also waiting for simplenewssheduler 6 version
#14
+1
#15
A first version created with coder and deadwood module.
They are a lot of code formating problem but my eclipse crash so I have no IDE to format all the code right now.
TODO:
- It's complety untesting...
- use shema function to create DB table in installation
- code formating
Please test it :)
#16
#17
thanks for the d6 port
but there is a bug
i selected a date,time in node edit page and saved it.
agian i edited the page the date is not set properly it is showing jan,1,1970,5:30 the default date
any configuration settings............
#18
subscribe
#19
Subscribing.
Sounds like a great module.
Any ETA on porting?
TIA
#20
I also want to subscribe. Unfortunately I have no programming skills to contribute anything... :-/
#21
@mnp thanks to test the module
I don't need this module for me directly, it's just for a friend. This is not a first priority for me so feel free to assign the task to you. I certainly work in this task in some weeks when i can test it with my friend on a real project (better to find bug).
Thanks to understand ;)
#22
I think I made some progress on this, please try it and give feedback.
#23
I made some changes and bug fixes since yesterday based on Simplenews 6.x branch.
Now it seems to be a working alpha release, but it needs review.
Please help test it, and report back!
#24
Subscribing. I'll try to test it out asap!
#25
Got this error...
warning: gmmktime() expects parameter 6 to be long, string given in /Volumes/arsenal/localhost/ntrst/sites/all/modules/simplenews_scheduler/simplenews_scheduler.module on line 141.
#26
Subscribing.
#27
Feed back
I've been testing it for over a week on Drupal 6.13 with a few members.
Works fine.
Next step is to add more members.
I think this should be uploaded so that more people can test.
#28
Subscribing
#29
Subscribing
Let's get this thing working ASAP! I am willing to test it and I am uploading it now!
#30
Hi,
I got a error message every time the newsletter is sent by schedule:
user warning: Duplicate entry 'newsletter/taegliches-update-test-de' for key 2 query: INSERT INTO url_alias (src, dst, language) VALUES ('node/323', 'newsletter/taegliches-update-test', 'de') in /srv/www/vhosts/my-page.com/subdomains/dev/httpdocs/modules/path/path.module on line 112.I don't see any duplicate entry, the url alias is set correctly to newsletter/taegliches-update-test-edition-X.
Greez,
Tobias
#31
I've installed the module from #23 and it does absolutely nothing for me... no settings, no extra options, no extra fieldset for scheduling, nothing...
Drupal 6.13
Simplenews RC6
Date 2.3
(and a whole bunch of other modules)
Is there anything I need to do to get this working? Anything I might have missed?
#32
Confirming that port works for D 6.13. Settings come when creating/editing a simplenews node by the way, not to anywhere else.
So, how about creting a new 6.x-dev branch?
Tx.
#33
Alpha-2 works for me with D6.13.
Here is a patch that embraces #256795: Allow simplenews cron to send nodes that anonymous users cannot access. If you apply the patch published there to Simplenews, newsletters can be send with the user rights of the publisher of the newsletter instead of anonymous, which is what you get using Cron.
The same logic holds for Simplenews Scheduler. If you have nodes with restricted access inside your newsletter, they will only get out on scheduled issues if the user rights are changed at rendering time.
#34
Oops, sorry, another change in my version got through that is not relevant here.
#35
Any chance of creating a dev release for those of us who don't play well with patches?
#36
I second that. Would be less intimidating for non-coders to start to download and test if it were listed as a 6x-dev module for download. Thanks for this module. I know a lot of people are waiting for this one. Who can I bribe to get this module completely ported faster?
Thanks very much,
Scott
#37
I sent an e-mail to dgtlmoon about a new 6.x-dev branch. I think this release is very alpha stage and much more people are needed to test it and report back to help the development. So keep an eye on this.
#38
Sorry for the delay, have just added the 6x dev branch, anyone want to take over some CVS work on this? i'll add you as a maintainer.. let me know, cheers.
#39
Thanks for the 6.x-dev branch, I would help with pleasure with CVS if you add me as a maintainer to speed this up a little bit.
#40
Hey sgabe, it says "sgabe does not have a CVS account."
#41
Have applied patches from comment #23 to latest 6x dev
#42
Ok well yesterday there was a 6x release on the project page but now it has gone away, i've rerun the CVS tag
cvs tag -b DRUPAL-6--1
5cvs update -r DRUPAL-6--1
but it looks like the tag is already there
i really hate CVS it's really getting in my way off getting any code out onto the drupal.org project, so..
@sgabe: are you confident in handling the 6x project stuff for this module?
#43
I took a look on the 6.x-dev branches, and I found both of them to be 5.x code so I committed the 6.x-1.0-alpha2 from #23 as 6.x-1.x-dev branch and applied the patch from #34. I think (if I am right...) with next cron run when drupal.org will update the download pages this will be fine and if you agree the 6.x-2.x-dev branch should be removed for now.
#44
Please note that this means the scheduler will only work with the 6.x-1.x-dev version of simplenews, but not the official release!
#45
This note is on the project page. The patch is not committed in the development snapshot (as far is I know) so I reworked it to be applicable on the current Simplenews 6.x-1.x-dev release.