.

ID:1903784
 
Jul 25 2015, 4:17 am (Edited on Apr 28 2020, 1:08 am)
Resolved
flick() previously didn't work with /icon datums, requiring a call to fcopy_rsc() first. Now the call is no longer required.
Applies to:Dream Daemon
BYOND Version:508
Operating System:Windows 7 Pro 64-bit
Web Browser:Chrome 44.0.2403.89
Status: Resolved (509.1300)
This issue has been resolved.
.
Jul 25 2015, 4:57 am
Are you seeing this behavior while using Dream Seeker, or the webclient? Or does it occur in both?
Jul 25 2015, 5:06 pm
This needs more information; I don't really have anything I can test. A test case would be helpful.
Jul 25 2015, 5:06 pm
Lummox JR changed status to 'Unverified'
Jul 28 2015, 1:47 pm
You might put together a test environment demonstrating the bug in action, then dump it in Dev Help to confirm whether it's a bug or not.

I've put together a bunch of bug reports in the past only to realize that I'm an idiot. This could be an instance of brain not fully engaged or misunderstanding of what's going on.

I usually run my bug reports by 2-3 people before I post them if I'm not able to get a clear handle on the issue.
Aug 11 2015, 12:54 pm
If it's not marked 'verified' or 'resolved' there's no indication Lummox has been able to even reproduce it, so I wouldn't get your hopes up.

It's actually marked 'unverified' which means Lummox tried to reproduce it and had no luck, you'll just have to wait and see what your latest test project yields for him when he gets around to looking at it.
Aug 11 2015, 12:57 pm
I'll take a look at this in a short while.
Aug 11 2015, 2:10 pm
Correction: Your file is no longer up. I'll need it to investigate your issue.
Sep 11 2015, 9:01 pm (Edited on Sep 11 2015, 9:08 pm)
http://files.byondhome.com/Flick/flickbug_src.zip

Link to a project showing this bug. Unless I'm missing something in using flick().

This is the entirety of the code:
mob
    icon = 'mob.dmi'
    verb
        FlickDMI()
            flick('smoke.dmi', src)

        FlickDynamicIcon()
            var/icon/I = icon('smoke.dmi')
            flick(I, src)

FlickDMI works fine, FlickDynamicIcon doesn't work at all.

EDIT: This is in DS. Didn't try it in the webclient yet.
EDIT2: Doesn't work in the webclient either.
Sep 14 2015, 8:24 am
And by the way, I can absolutely verify the accuracy of the title of this post...
Sep 14 2015, 10:04 am
I'll be looking into this today.
Sep 14 2015, 12:41 pm
I've moved this to Feature Requests because I found the cause, and IMO it's not a bug, just a missing feature.

The problem is that you're flicking with an /icon datum, not an actual cached icon. I'll add support for that.
Sep 14 2015, 12:53 pm
Lummox JR resolved issue with message:
flick() previously didn't work with /icon datums, requiring a call to fcopy_rsc() first. Now the call is no longer required.