Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Is this module still maintained and do you plan to port to D7? Thanks
Comment | File | Size | Author |
---|---|---|---|
#21 | multiping-7.x-dev.patch | 37.38 KB | openWeb |
#20 | 7.x-dev.patch | 37.38 KB | openWeb |
#12 | ping.zip | 2.93 KB | Alex Bacart |
#4 | multiping.zip | 7.98 KB | andypost |
Comments
Comment #1
voxpelli CreditAttribution: voxpelli commentedI will probably port it - but unfortunately have no timeline for it - I have a lot of other more prioritized modules.
Comment #3
willmoy CreditAttribution: willmoy commentedFair enough, thanks for the quick response. It's conceivable I might have some free time in a couple of months and will try to help out if so.
Comment #4
andypostThis is a initial port.
Admin UI works, ping from UI works too.
Pinging logic should be refactored
1) _multiping_checkpings() - should accept a parameter of changed entity
2) _multiping_doping() - should accept second parameter a URL to ping
Comment #5
voxpelli CreditAttribution: voxpelli commentedCan we have a patch? Also - change status to "needs review" when you feel you're ready for a review.
Comment #6
andypost@voxpelli I just wanna to know maintainer's opinion before changing signature of main functions of the module.
I think that _multiping_doping($service, $options) should implement a ping Only and _multiping_checkpings($entity) should loop through services|settings and call _doping()
probably you have different vision so I'd like to know you opinion!
Comment #7
voxpelli CreditAttribution: voxpelli commentedI think that we should refactor multiping. Although I'm also generally in favor of separating the refactoring from the porting so that both the D6 and the D7 version can get the advantages from the refactoring.
What should be done for this specific module I currently have no opinion on. I have a roadmap on http://drupal.org/project/multiping which I currently haven't had time to start on.
If you're able to separate out the refactoring into its own issue and provide patches for both D6 and D7 then that would be great. If not then I think I'll accept any patch anyway as long as it works.
Comment #8
andypost@voxpelli I suppose it's better to start D7 version as 're-factored' because core have a lot of changes and then provide a upgrade path from D6.
Main reasons is about taxonomy that been slightly changed
- taxonomy vocabs are using machine_name so IDs is a thing of past
- taxonomy mostly a reference field and could be attached to any entity
- taxonomy now have only taxonomy/term/ID/feed
Also I think that better to start D7 branch to get more feedback from community before releasing a stable
Comment #9
voxpelli CreditAttribution: voxpelli commented@andypost: I think it would be very much possible to have similar version on D6 and D7 and I think that would be fairest to everyone. Of course an upcoming version can be unique to D7. But as as said - something is better than nothing, but don't change things just for the sake of changing - then we would have to either support two different modules or discontinue support for the D6 version.
Comment #10
sillygwailoComment #11
Taxoman CreditAttribution: Taxoman commentedSubscribing
Comment #12
Alex Bacart CreditAttribution: Alex Bacart commentedHi there!
This is my Drupal7 ping solution.
Ping module
Comment #13
Taxoman CreditAttribution: Taxoman commentedHm. Why is that file not uploaded here but resides on a russian server?
Comment #14
Alex Bacart CreditAttribution: Alex Bacart commentedI don't know. That's why gave a link to the russian drupal community site.
Comment #15
voxpelli CreditAttribution: voxpelli commentedIf someone wants to take over as the maintainer of this module - then just say so and we'll try to arrange it.
Comment #16
SpleshkaI can help you with porting module to 7.x. Just give me access to git repository and I'll do the rest.
Comment #17
andypost@Spleshka just provide a patch :) or use a sandbox
Comment #18
Spleshka@andy, but I wanna to be a co-maintainer :)
Comment #19
openWeb CreditAttribution: openWeb commentedI am currently working on a port to d7...
will post patch againt the master-branch later this day
Comment #20
openWeb CreditAttribution: openWeb commentedHere we go... Patching master -> 7.x-dev (if you like to)...
"coder" gives no more errors or warnings, should overlook _multiping_checkpings to use the dbng-api, but could not do that in the time...
Comment #21
openWeb CreditAttribution: openWeb commentedSetting correct status and new name for patch...
Comment #22
SpleshkaopenWeb, try db_merge() instead of db_update() and db_insert(). It's more useful function.
Comment #23
voxpelli CreditAttribution: voxpelli commentedI'm not going to have time to look into this - anyone want to be added as a co-maintainer?
Comment #24
SpleshkaPlease add me. I'll make stable D7 release.
Comment #25
andypostAny reason to make this port many times? My patch in #3 mostly the same as @openWeb provides.
It uses core's approach to menu loaders and comment suggests to change a bit parameters for internal functions (prefixed with underscore)
use parameter - arg() as broken way.
same, take my code from #3
do not change $form to $return - this function is a form builder
how many times you make direct queries to this table? I dont think that there's a lot of services in table so drupal_static is good pattern to go but dont forget to clean em
a lot of trailing whitespace errors all over the patch, and there's some places with wrong indent
Comment #26
andypost@Spleshka if you wanna maintain this please provide a patch
Comment #27
voxpelli CreditAttribution: voxpelli commented@anypost: Added you as a co-maintainer - if you need more co-maintainers then contact me and when you feel like you're ready to become a full maintainer then contact me.
Comment #28
andypostInitial version commited
7.x-1.x-dev release tarball upcoming
Things todo:
Upgrade path for taxonomy
- 6.x stores term->tid in service->voc but 7.x should save config per node-type to limit ping by term
- currently commited stub just hides taxonomy config in admin
Refactoring the ping model
- tremendous queries executed for each ping service just to find a changed and created nodes
- change ping to different signature _multiping_do_ping($service, $node)
- use queue to store jobs to ping
- depricate_multiping_checkpings() or change for queue generation
Comment #29
andypostI'm going to start 7.x-2.x branch with breaking functionality into
global settings
- manage services
- link services to node types, probably just node-type checkboxes to allow per node settings
- ping frontpage on allowed node-types
per node settings
- node type form settings: pingable, limits
- node edit form: checkbox to ping on insert|update
Comment #30
Plazik CreditAttribution: Plazik commentedSimilar D7 module http://drupal.org/project/simpleping
Comment #31
revagomes CreditAttribution: revagomes commentedPing :P
I'm just posing a reminder to the maintainers to let people interested in this module to know when will be a D7 release version available to download. The maintainers can use this issue here to keep us followers posted on things like the progress made, if they needs a certain feature to be tested or when the next release is planned to be made.
Comment #32
anandaqt CreditAttribution: anandaqt commentedI think Multiping for drupal 7 is conflict with cron, ckeditor which make the cron could not run. The Ckeditor become dead white screen when save or update an old post. More importance it not ping when i save or update a post. Test drupal 7.38
Comment #33
voxpelli CreditAttribution: voxpelli commentedJust a headsup here – I'm going to be looking into some updates to this module, either as a new 7.x-1.x version or as a 7.x-2.x version.
My primary focus will be to enable modern pinging mechanisms like Pubsubhubbub and to make it possible to easily integrate other parts of Drupal into such pinging and also to provide some such integrations.
I may also look into the legacy pinging as well as adding other modern pinging mechanisms like Webmentions.
Comment #34
voxpelli CreditAttribution: voxpelli commentedSpeaking of Webmentions – looks like https://www.drupal.org/project/vinculum has that and support many of the other ping types + is extendable. So even though I've maintained this module before, it looks like one would likely be better of moving the XML-RPC pinging that is at the core of this module to that one instead – and for me to make a patch for Pubsubhubbub to that module rather than as an extension of this one.
So – probably a change of plan to that unless it proves to not work.
Comment #36
voxpelli CreditAttribution: voxpelli commentedI've pushed a new major version of this module for Drupal 7 and released an alpha of that so I'm going to close this as fixed.
All remaining items to track before a stable 7.x-2.0 release can be made can be found in #2616616: Tracking issue for 7.x-2.0
Comment #37
jonathan1055 CreditAttribution: jonathan1055 commentedHey that's great work. Thank you.
Comment #38
andypostFiled follow-up ;) #2616764: Port Multiping to Drupal 8