MMBase

mm:changed does not work in relatednodes with sortorder on role

Details

  • Type: Bug Bug
  • Status: Reopened Reopened
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.8.2
  • Fix Version/s: 2.0.0
  • Component/s: Taglib
  • Description:
    Hide
    original bugid: #6906
    <mm:relatednodescontainer type="news' role="posrel">
      <mm:sortorder field="posrel.pos" />
      <mm:relatednodes>
        <mm:changed>
          *** does not work properly ***
        </mm:changed>
      </mm:relatednodes>
    </mm:relatednodescontainer>

    as oppsoed to doing this with a <mm:related> tag
    Show
    original bugid: #6906 <mm:relatednodescontainer type="news' role="posrel">   <mm:sortorder field="posrel.pos" />   <mm:relatednodes>     <mm:changed>       *** does not work properly ***     </mm:changed>   </mm:relatednodes> </mm:relatednodescontainer> as oppsoed to doing this with a <mm:related> tag

Activity

Hide
Michiel Meeuwissen added a comment - 2009-07-31 16:39
What does it do improperly, how was it supposed to be?
Show
Michiel Meeuwissen added a comment - 2009-07-31 16:39 What does it do improperly, how was it supposed to be?
Hide
Pierre van Rooden added a comment - 2009-07-31 16:58
Say, you have posrels with 1, 1, 2, 2, and 3.

If you list these with a related list, and set sortorder to 'posrel.pos', the <mm:changed> tag will trigger thrice: one teh first 1, on the first 2, and on the 3.
.
If you list these with a relatednodes list, and set sortorder to 'posrel.pos', the <mm:changed> tag will trigger five times: once for every record.

Maybe that is 'correct' behavior, but it is not very intuitive.
Show
Pierre van Rooden added a comment - 2009-07-31 16:58 Say, you have posrels with 1, 1, 2, 2, and 3. If you list these with a related list, and set sortorder to 'posrel.pos', the <mm:changed> tag will trigger thrice: one teh first 1, on the first 2, and on the 3. . If you list these with a relatednodes list, and set sortorder to 'posrel.pos', the <mm:changed> tag will trigger five times: once for every record. Maybe that is 'correct' behavior, but it is not very intuitive.
Hide
Michiel Meeuwissen added a comment - 2009-07-31 17:03
Hmm, yes. I don't know. It's the node that changes every time, isn't it?
Show
Michiel Meeuwissen added a comment - 2009-07-31 17:03 Hmm, yes. I don't know. It's the node that changes every time, isn't it?
Hide
Michiel Meeuwissen added a comment - 2009-07-31 17:04
Any how, I don't realy care how it is supposed to work. If find mm:changed confusing anyhow. If you allow me, I simply assign it to yourself :-)
Show
Michiel Meeuwissen added a comment - 2009-07-31 17:04 Any how, I don't realy care how it is supposed to work. If find mm:changed confusing anyhow. If you allow me, I simply assign it to yourself :-)
Hide
Pierre van Rooden added a comment - 2009-07-31 17:12
The issue is that the field 'posrel,pos' is not available for mm:change to check when you use a relatednodes list (because it is not in the resultset).
So it triggers always.
Show
Pierre van Rooden added a comment - 2009-07-31 17:12 The issue is that the field 'posrel,pos' is not available for mm:change to check when you use a relatednodes list (because it is not in the resultset). So it triggers always.
Hide
Michiel Meeuwissen added a comment - 2009-07-31 17:24
Yes probably, but what can we do. it will be quite involving to change that. On the other hand I do'nt think the current behaviour actually wrong. I think mm:changed on nodelist is simply useless.

Show
Michiel Meeuwissen added a comment - 2009-07-31 17:24 Yes probably, but what can we do. it will be quite involving to change that. On the other hand I do'nt think the current behaviour actually wrong. I think mm:changed on nodelist is simply useless.
Hide
Michiel Meeuwissen added a comment - 2009-07-31 17:25
I think indeed it will be quite a shift, 'nodelist' must contain some kind of combined node or so. To involving for a minor release. Rescheduled for 2.0.
Show
Michiel Meeuwissen added a comment - 2009-07-31 17:25 I think indeed it will be quite a shift, 'nodelist' must contain some kind of combined node or so. To involving for a minor release. Rescheduled for 2.0.
Hide
Pierre van Rooden added a comment - 2009-08-03 08:52
It is not entirely useless, as long as you use a field in mm:sortorder that mm:changed can access.
Maybe we need to change the documentation.

Show
Pierre van Rooden added a comment - 2009-08-03 08:52 It is not entirely useless, as long as you use a field in mm:sortorder that mm:changed can access. Maybe we need to change the documentation.

People

Dates

  • Created:
    2006-10-26 14:46
    Updated:
    2009-08-03 08:52