Safe trash/shadow assets bug

I logged a bug last year, which was apparantly fixed:


http://bugs.matrix.squiz.net/view_bug.php?bug_id=3812



However, we're running 3.24.1 and it doesn't seem to be fixed on that version. Neither can I find any changes in the CVS that indicate a change. Can anyone point me in the direction of what files were changed?

I cant get to the public CVS URL at the moment, but it was core/include/asset_manager.inc


Commit info is for 3.24:

Revision 1.738.2.4

Wed Aug 5 06:34:47 2009 UTC (9 months, 1 week ago) by ewang

Branch: mysource_3-24-0-branch

Changes since 1.738.2.3: +5 -2 lines

Diff to previous 1.738.2.3 , to branch point 1.738

Bug fix #3812 – Getting shadow links shoule also respect link types. If we try to get notice links of a shadow asset, we don't expect type 1 links.

I've just been reminded about this issue.


Have checked the diff for that bug fix via the CVS and the code on our system, and it is definitely there. This change probably did fix something (we see less warnings about LDAP users now), but it hasn't fixed the actual bug; that you can't trash LDAP users when safe trash is enabled.



Can the bug therefore be re-opened please?

Please submit a new bug report if you still have an issue. We'll merge bug reports together if required.