First of all, thank you!
I am using nodequeue + views to produce my content; using nodequeue position id in views sort criteria. Sorting works on the views side; whenever I change the order of elements in my nodequeue the views reflects this.
I've also setup ddblock to show on my page, with number pagers.
Problem: Whatever the sort of the nodequeue or views sort criteria, ddblock sorts the content by itself, it does not respect the order of the elements coming from views block.
There is a configuration setting in the block configuration page (random, asc, desc) but it works as if it gets the rows from the views block unordered and later applies a sorting on top of it; which makes ordering of nodequeue elements useless.
Any hint?
Comment | File | Size | Author |
---|---|---|---|
#21 | ddblock.module1.txt | 81.54 KB | ppblaauw |
#20 | ddblock.module.txt | 81 KB | venusrising |
#13 | ddblock.module.rej_.txt | 3.27 KB | venusrising |
#2 | ddblock-add-order-of-supplied-content-can-now-be-used-397018-1.patch | 2.72 KB | ppblaauw |
Comments
Comment #1
serkan.arikusu CreditAttribution: serkan.arikusu commentedHi again,
I've solved the problem by removing the line
asort($content);
in ddblock.module.I think this overrides the views sort criteria.
Maybe we should have a "None" option in ddblock configuration page, which does not do anything?
Comment #2
ppblaauw CreditAttribution: ppblaauw commentedThanks for reporting.
Next version will have a "None" option for the order.
None for using the original content order.
The attached patch can be used to use the original content order. e.g. from nodequeue.
Please let me know if this works.
Comment #3
serkan.arikusu CreditAttribution: serkan.arikusu commentedIt works, thank you...
Comment #4
mrfelton CreditAttribution: mrfelton commentedworks for me too. Thanks
Comment #5
ppblaauw CreditAttribution: ppblaauw commentedClosing issue, patch will be applied in next version of the dynamic display block module.
Comment #6
coderb CreditAttribution: coderb commentedHi,
Thanks for the module and the sort patch, but I have a question regarding the sorting.
For my ddblock content type, I am using a cck imagefield which allows unlimited uploads.
The reason I am using this imagefield, is because it provides a thumbnail and drag-and-drop sort option of all uploaded images.
The ddblock module, with sort patch, and set to sorting: 'none', displays all images in the defined folder in order of upload.
If I then do the drag-and-drop sort of my uploaded images, it has no effect on the display order of the slideshow.
Is there an easy way to pick up that order? Or perhaps there is an easier way to for me to handle this?
thanks for any advice.
Comment #7
zdean CreditAttribution: zdean commentedIs it possible this patch could be applied to the RC version and then published? I am unable to upgrade to the dev (breaks my ddblock) and not able to apply the patch myself. Any help is appreciated.
Thanks.
Comment #8
zdean CreditAttribution: zdean commentedI was able to apply the patch using this tutorial:
http://drupal.org/node/60818
Comment #9
Anonymous (not verified) CreditAttribution: Anonymous commentedThanks for the patch. This helped to solve my problem with custom sorting.
Comment #10
Danny_Joris CreditAttribution: Danny_Joris commentedHello. First of all, thanks for the patch.
I just completely configured a ddblock slider using the 6.x-1.0-rc6 version and now I stumble on this issue. I don't feel like upgrading to the dev version at this stage, so I will probably tweak the patch in manually.
It would be really great if this very basic functionality gets a fix in a next release candidate. The current RC is dated February 26 2009 (!).
Comment #11
venusrising CreditAttribution: venusrising commentedThis patch fails on the RC version. Is any way to get a patch that will will with the RC version and not DEV since the RC is from 2009 and we really dont want to run a dev module on a production site and not being able to re-order the DD Block makes it hard to use.
Comment #12
ppblaauw CreditAttribution: ppblaauw commentedThe patch should work with RC6.
The patch is relatively small. It can be applied manually in a few minutes.
The dev version is for test purposes and should not be used on websites for this module.
Comment #13
venusrising CreditAttribution: venusrising commentedWeird-
I get the following fails, see attached doc
Comment #14
venusrising CreditAttribution: venusrising commentedHi There, cannot get the patch to apply to the RC, see above
Comment #15
ppblaauw CreditAttribution: ppblaauw commentedThe patch is relatively small.
It can be applied manually in a few minutes.
I suggest you do this.
Set status to postponed
Comment #16
venusrising CreditAttribution: venusrising commentedThis is not easy to apply as the stable module code does not match the patch module code for instance the code in the patch at line 31-41 does not coincide with the module code so it is not easy to see where to put it which seems like why the patch failed to begin with. Is there any way to clarify this or post a module file update to this post as it is difficult to use the module when the nodes cannot be ordered.
Thanks!
Comment #17
ppblaauw CreditAttribution: ppblaauw commentedCan you attach your ddblock.module file.
I will patch it for you
Comment #18
Danny_Joris CreditAttribution: Danny_Joris commentedWhy no commit?
Comment #19
ppblaauw CreditAttribution: ppblaauw commented#18 busy, priorities.
Comment #20
venusrising CreditAttribution: venusrising commentedThanks, that is so nice of you. I had to add txt to the end as upload would not allow mod files.
Comment #21
ppblaauw CreditAttribution: ppblaauw commented#20 Attached the modified ddblock.module file
Rename again to ddblock.module
Hope this helps you further, please let me know.
Comment #22
narendrak CreditAttribution: narendrak commentedI was facing orderby problem with slideshow ddblock. i found that in views_slideshow_ddblock.module
function template_preprocess_views_slideshow_ddblock(
// sort the result using ddblock, none for using order of view
if (!empty($rows)) {
switch ($options['settings']['order']) {
case 'random':
shuffle($rows);
break;
case 'asc' :
asort($rows);
break;
case 'desc':
rsort($rows);
break;
case 'none':
break;
}
just comment this code . it will take views sort values.
Comment #23
pbull CreditAttribution: pbull commentedSuccessfully applied ppblaauw's patch (http://drupal.org/node/397018#comment-1355100) against ddblock 6.x-1.0-rc6 with no troubles.
Comment #24
ppblaauw CreditAttribution: ppblaauw commented#22 please leave original title, this issue is not for the views_slideshow_ddblock module but for the ddblock. Would make it confusing for other users.
In views_slideshow_ddblock the none option is already added in the settings page.
You don't need to apply any patch and don't need to comment the code, just choose none in the configuration page of the slideshow (views display style settings)
set to patch (to be ported) till it is committed.
Comment #25
mmachina CreditAttribution: mmachina commentedpatch worked beautiful...
Comment #26
claar CreditAttribution: claar commentedPatch worked great -- thanks, ppblaauw. Will be a great improvement when committed.
Comment #27
jonwithhtop CreditAttribution: jonwithhtop commentedWORKED LIKE A DREAM!
Comment #28
claar CreditAttribution: claar commentedComment #29
msathesh CreditAttribution: msathesh commented@ppblaauw patch works great.. thanks..