Hello!

I just discovered that when using subforums, the link that is created to display that a new post is written there is like this: http://forum/53%23new

That does not work very well. =)

Best regards,
Ao

CommentFileSizeAuthor
#3 af7_1109820.patch1.07 KBtroky
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

troky’s picture

Where exactly?

Aonoa’s picture

On the first "forum" page, if you go to /forum.

Container
-- Forum_1
---- Subforum of Forum_1 <-- this one has the broken link when new posts are written.

Best regards,
Ao

troky’s picture

Status: Active » Needs review
FileSize
1.07 KB

This one should fix it. Test and report back.

Michelle’s picture

Shoot, looks like this is broken in D6 as well. Could you shoot this over there once it's settled in D7? I'm not in a position to fix it right now and don't want to forget it. That's a pretty bad bug... Looks like Alpha 5 is going to come faster than I thought. :(

Michelle

Michelle’s picture

There's more problems here. I noticed you didn't fix the subsubcontainer link in your patch... which got me looking at subsubcontainers... which got me realizing that subsubcontainer aren't calculating child posts... which got me realizing that subsubforums aren't, either. So I need to work on the logic more. :(

Michelle

Michelle’s picture

Actually... I just realized it's working as intended but how it was intended is problematic. I'm going to start a new issue because this needs to get hashed out before I make changes and isn't quite the same as the bug that was posted here. So let's get that actual bug fixed first and then worry about the rest.

Michelle

Michelle’s picture

Status: Needs review » Reviewed & tested by the community

Ok, so I committed your patch to D6 with the addition of the line to fix the problem in the subcontainer theming. That's useless right now since that will always be 0 but good to have it fixed so it's working when the underlying logic starts working.

Once this is in D7, this issue can be marked fixed. I'm going to go start a new one for the other issue right now.

Michelle

troky’s picture

Status: Reviewed & tested by the community » Fixed

I've noticed problems from #5 but it was too late to examine them so I just made quick fix for this issue only.

Committed to D7.

Aonoa’s picture

Using the latest D7 snapshot from today I still get subforum links like this: http://forum/53%23new
It does not include the domain name in the link.

Best regards,
Ao

Aonoa’s picture

Status: Fixed » Active
troky’s picture

Status: Active » Fixed

Check date and time of my message and compare to 7.x-dev release. It's not packed yet.
So, patch it manually or wait few more hours.

Aonoa’s picture

Aha, I did not really check the date/time beyond the day.. Sorry. The patch when applied does indeed fix the issue. =)

Thank you!

Best regards,
Ao

troky’s picture

Thanks for testing it ;)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.