Welcome Guest! To enable all features please Login or Register.

Notification

Icon
Error

Topic Tags: Sueetie Data Core, Tagging
3 Pages<123>
cja100
#11 Posted : Friday, February 25, 2011 11:44:13 AM(UTC)


Rank: Lieutenant

Joined: 3/18/2010(UTC)
Posts: 31

Only one user had the trouble after i fixed it, and i havnt heard from them since.
Dave Burke
#12 Posted : Friday, February 25, 2011 3:35:50 PM(UTC)


Rank: Administration

Joined: 12/7/2008(UTC)
Posts: 1,267
Man
United States

Thanks: 37 times
Was thanked: 69 time(s) in 68 post(s)
Thanks for following up on this. Good to hear it's been quiet.

-Dave
cja100
#13 Posted : Saturday, February 26, 2011 12:27:31 AM(UTC)


Rank: Lieutenant

Joined: 3/18/2010(UTC)
Posts: 31

He may of just given up lol.

I created an entry for each post and topic in the Sueetie_Content table that was missing one, but now I get an error for the same topic.

Line 182 in TagControl.cs = var _userIDs = TagSueetieForumTopic.SueetieUserIDs.Split('|').Select(n => int.Parse(n)).ToList();

Its having trouble parsing the topic because for some reason the ConcatForumTopicUsers function is passing a value with an extra | at the end.

So i edited the query slightly

Code:
    select distinct s.UserName, s.UserID
    FROM    dbo.[yaf_Message] m
        JOIN dbo.yaf_topic t
            ON t.TopicID = 10
            JOIN dbo.yaf_User u ON
                m.UserID = u.UserID
                JOIN dbo.Sueetie_users s
                    ON s.Username = u.Name
                    where m.TopicID = 10 and u.BoardID = 1


All the users exist in the sueetie_Users and yaf_User tables.

SueetieUserIDs value =
Code:
"8|8|8|8|8|8|8|11|11|11|11|11|14|14|14|14|14|16|16|16|16|16
|16|16|16|16|16|16|16|18|20|21|22|22|22|22|22|25|26|27|27|27|27
|34|40|43|44|49|56|56|75|75|76|89|89|89|89|89|97|97|97|97|97|97|97|97|97|97|97|"


Code:



Line 180: if (TagSueetieForumTopic != null)
Line 181: {
Line 182: var _userIDs = TagSueetieForumTopic.SueetieUserIDs.Split('|').Select(n => int.Parse(n)).ToList();
Line 183: foreach (int _userID in _userIDs)
Line 184: {

Source File: C:\inetpub\atomo\Sueetie.Controls\Controls\TagControl.cs Line: 182

Stack Trace:


[FormatException: Input string was not in a correct format.]
System.Number.StringToNumber(String str, NumberStyles options, NumberBuffer& number, NumberFormatInfo info, Boolean parseDecimal) +10161091
System.Number.ParseInt32(String s, NumberStyles style, NumberFormatInfo info) +207
Sueetie.Controls.TagControl.<IsUserTagEditor>b__0(String n) in C:\inetpub\atomo\Sueetie.Controls\Controls\TagControl.cs:182
System.Linq.WhereSelectArrayIterator`2.MoveNext() +110
System.Collections.Generic.List`1..ctor(IEnumerable`1 collection) +258
System.Linq.Enumerable.ToList(IEnumerable`1 source) +44
Sueetie.Controls.TagControl.IsUserTagEditor() in C:\inetpub\atomo\Sueetie.Controls\Controls\TagControl.cs:182
Sueetie.Controls.TagControl.OnInit(EventArgs e) in C:\inetpub\atomo\Sueetie.Controls\Controls\TagControl.cs:260
System.Web.UI.Control.InitRecursive(Control namingContainer) +143
System.Web.UI.Control.AddedControl(Control control, Int32 index) +271
YAF.Pages.posts.Page_Load(Object sender, EventArgs e) in C:\inetpub\atomo\Web\Forum\pages\posts.ascx.cs:274
System.Web.Util.CalliHelper.EventArgFunctionCaller(IntPtr fp, Object o, Object t, EventArgs e) +25
System.Web.Util.CalliEventHandlerDelegateProxy.Callback(Object sender, EventArgs e) +42
System.EventHandler.Invoke(Object sender, EventArgs e) +0
System.Web.UI.Control.OnLoad(EventArgs e) +132
System.Web.UI.Control.LoadRecursive() +66
System.Web.UI.Control.LoadRecursive() +191
System.Web.UI.Control.LoadRecursive() +191
System.Web.UI.Control.LoadRecursive() +191
System.Web.UI.Control.LoadRecursive() +191
System.Web.UI.Control.LoadRecursive() +191
System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint) +2428

Any ideas?

Edited by user Saturday, February 26, 2011 12:29:26 AM(UTC)  | Reason: Not specified

Dave Burke
#14 Posted : Saturday, February 26, 2011 10:05:40 AM(UTC)


Rank: Administration

Joined: 12/7/2008(UTC)
Posts: 1,267
Man
United States

Thanks: 37 times
Was thanked: 69 time(s) in 68 post(s)
Christopher,

You provided such excellent detail that I had to come up with the solution for you, and I think I have. This is a REALLY long Forum Topic!

Code:
8|8|8|8|8|8|8|11|11|11|11|11|14|14|14|14|14|16|16|16|16|16
|16|16|16|16|16|16|16|18|20|21|22|22|22|22|22|25|26|27|27|27|27
|34|40|43|44|49|56|56|75|75|76|89|89|89|89|89|97|97|97|97|97|97|97|97|97|97|97|


One thing you didn't mention about the function is that it maxes out at 200, and the user string for this topic exceed 200 characters. If you jack up the function return nvarchar(200) value to, say, nvarchar(500), it may be your solution. But then again, you did what I would have done, use a "distinct" to the query, in which case you should have come up with something like

Code:
8|11|14|16|...


which falls below the nvarchar(200) limit, so maybe not.

Let me know if using the original function and increasing the nvarchar(200) does it please. I'll review this for v3.1 based on your input. I know your excellent community uses Sueetie v2.0 (the one I looked at anyway. Great site and theme, btw!), but you'd be helping everyone out if you could follow-up.

Thanks much,
Dave
cja100
#15 Posted : Saturday, February 26, 2011 10:46:19 AM(UTC)


Rank: Lieutenant

Joined: 3/18/2010(UTC)
Posts: 31

Looks like that fixed it, I changed the limit to 500. Thanks for the help.
Dave Burke
#16 Posted : Saturday, February 26, 2011 11:59:49 AM(UTC)


Rank: Administration

Joined: 12/7/2008(UTC)
Posts: 1,267
Man
United States

Thanks: 37 times
Was thanked: 69 time(s) in 68 post(s)
Christopher,

Thank you for confirming the nvarchar() increase fixed it for you. Like I said, I'll be looking into this for the next release of Sueetie coming soon.

-Dave
Dave Burke
#17 Posted : Tuesday, March 8, 2011 7:03:32 AM(UTC)


Rank: Administration

Joined: 12/7/2008(UTC)
Posts: 1,267
Man
United States

Thanks: 37 times
Was thanked: 69 time(s) in 68 post(s)
Christopher,

Fix complete. Here are the details, along with the user function which you will be able to use with your version of Sueetie.

Thanks,
Dave
styx31
#18 Posted : Monday, August 8, 2011 9:53:28 PM(UTC)


Rank: Lieutenant

Joined: 12/18/2008(UTC)
Posts: 111


Was thanked: 1 time(s) in 1 post(s)
One of our moderators seems to have been able to reproduce the problem.

If you open an existing thread, then move one post to a new thread (so creating it), then the new thread will throw a NRE.

I think sueetie does not handle topic split. Also perhaps it should handle null content quietly ?
Dave Burke
#19 Posted : Tuesday, August 9, 2011 1:54:15 AM(UTC)


Rank: Administration

Joined: 12/7/2008(UTC)
Posts: 1,267
Man
United States

Thanks: 37 times
Was thanked: 69 time(s) in 68 post(s)
Originally Posted by: styx31 Go to Quoted Post
I think sueetie does not handle topic split. Also perhaps it should handle null content quietly ?


No, split topics aren't integrated into the Sueetie Data Core. Will try to add that...

Thanks,
Dave

Update: Added to Roadmap To-Do Specifics List.

Edited by user Tuesday, August 9, 2011 4:24:31 AM(UTC)  | Reason: Not specified

styx31
#20 Posted : Tuesday, August 9, 2011 6:01:19 AM(UTC)


Rank: Lieutenant

Joined: 12/18/2008(UTC)
Posts: 111


Was thanked: 1 time(s) in 1 post(s)
Great ! Thanks. I'll patch atomo until then.
Rss Feed  Atom Feed
Users browsing this topic
Guest
3 Pages<123>
Forum Jump  
You cannot post new topics in this forum.
You cannot reply to topics in this forum.
You cannot delete your posts in this forum.
You cannot edit your posts in this forum.
You cannot create polls in this forum.
You cannot vote in polls in this forum.

Powered by YAF 1.9.6.1 | YAF © 2003-2013, Yet Another Forum.NET
This page was generated in 0.376 seconds.
Follow Sueetie on Twitter
Copyright © 2008-2012 Sueetie LLC. All rights reserved.