Digital Point Spy

[James]

Well for the deletion events, we are really just piggybacking XF's moderator log... so if XF logs it as a deleted post in it's moderator log, that's what it will show up as in the Spy.
Maybe someday, but don't really have a setup to really allow it, so not at the moment.

http://xenforo.com/community/posts/282458/ Mike fixed that bug - hopefully next version it should show the correct delete

 

[Mikey]

I think this would look better if you did it more like the news feed / alerts. Whereas you display a bunch of rows of data which is controled by handlers for each content type. Instead of the table layout you have.

So for example. Instead of...

View attachment 20882

Have something similar to...

View attachment 20883

Smething like this to make this in essence, the recent activity but with ajax updating would be pretty awesome

As it is now is fine, but http://thegeekdistrict.com/spy doesn't look as nice as http://thegeekdistrict.com/recent-activity (formatting-wise)

 

[digitalpoint]

BTW - you should take out the "height:50px" line in spy.css... not sure exactly what I was thinking there. lol

I've updated it for the next version, but not really worth doing a release just for that.

 

[mrGTB]

You've also got a problem there with breaking the style layout in SPY. Unless it's an issue related to just that custom style you use?

 

[digitalpoint]

You've also got a problem there with breaking the style layout in SPY. Unless it's an issue related to just that custom style you use?

View attachment 20907

It's related to the post I made right before yours.

 

[mrGTB]

Ah, sorry. Didn't spot you mention it there.

 

[Mikey]

BTW - you should take out the "height:50px" line in spy.css... not sure exactly what I was thinking there. lol

I've updated it for the next version, but not really worth doing a release just for that.

done, thanks

 

[Mikey]

Possibly a bug?

I have a thread called "^<v" and in the spy it didn't seem to parse the thread title properly.

http://thegeekdistrict.com/spy/

 

[James]

Possibly a bug?

I have a thread called "^<v" and in the spy it didn't seem to parse the thread title properly.

http://thegeekdistrict.com/spy/

Open library/DigitalPointSpy/ControllerPublic/Spy.php

Find (line 60):

$item['title']

Replace:

htmlentities($item['title'])

Does that work?

You might want to use XenForo coding standards and use XenForo_Template_Helper_Core::callHelper('striphtml') though.

Let me know if this works

 

[digitalpoint]

Should probably toss one on forum_title for good measure... not sure why I put one on the username, but nothing else. Probably had something to do with me coding it at 3:30 am.

 

[digitalpoint]

Oh, now I remember why I added the height:50px... it's because of the inability for jQuery to calculate the height on it's own for some reason (leads to the weird animation).

I'll see if I can come up with a better way...

 

[James]

Perhaps put the JS into a XenForo template and use {xen:calc}, but I think that'd require you to be utilising custom style properties for it to work.

 

[digitalpoint]

Actually, I got it squared away... if you change what was originally "height: 50px", to "width:100%", it seems to work...

.discussionListItems .itemWrapper {
	display: none;
	width: 100%;
}

Had to do with jQuery temporarily making the block an inline-block when it calculates the height, which gave it a width of 0, so everything was way to big vertically... forcing it out to 100% width works (even while in inline-block mode).

 

[Floris]

Reported posts seem to be in the list, and visible to regular users. Not really desirable I think.

 

[Digital Doctor]

Reported posts seem to be in the list, and visible to regular users. Not really desirable I think.

Yea. I was in the process of reporting that.

There is also no link to the post that is being reported.
I also think it would be more valuable to put the name of the person being reported *NOT* the reportee !

 

[digitalpoint]

The reported posts showing up is intentional... as is the reporter (as opposed to the reportee), since the action was taken by the reporter... but if you don't want it there, you can comment out this line from the Listener/LoadClassDataWriter.php file:

//			$extend[] = 'DigitalPointSpy_DataWriter_Report';

 

[Digital Doctor]

I'm not sure people reporting posts will be encouraged to do so when it is the reportee not the offendee that is being put on trial. I guess it just depends on the atmosphere of your forum. I feel the main focus of reporting posts is to judge the post, not the reporter.
IMO.

 

[Mikey]

Open library/DigitalPointSpy/ControllerPublic/Spy.php

Find (line 60):

$item['title']

Replace:

htmlentities($item['title'])

Does that work?

You might want to use XenForo coding standards and use XenForo_Template_Helper_Core::callHelper('striphtml') though.

Let me know if this works

Worked.

htmlentities() works fine for me until an 'official' patch is released

thanks!

 

[Alien]

I have been unable to get anything to display just yet...

I also don't appear to have a dp_spy table?

Several of the other sites that installed this appear blank to me too, any idea what might be going on?

http://forums.wtf.com/spy/

 

[digitalpoint]

Are you certain there is no dp_spy table? I would suspect you would be getting a SQL error when it tried to read it (make sure you aren't checking for xf_spy)...