This link appears when comments are enabled but not available to anonymous users - we need a test to ensure it appears correctly.

CommentFileSizeAuthor
#7 comment-register-296484.patch1.11 KBjonskulski
#4 issue-296484.patch1.06 KBlilou
#2 anon_user.patch845 bytesswentel
#1 anon_user.patch718 bytesswentel

Comments

swentel’s picture

swentel commented
StatusFileSize
new anon_user.patch718 bytes
swentel’s picture

swentel commented
Status: Active » Needs review
StatusFileSize
new anon_user.patch845 bytes

Changed hardcoded nid, looks better.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Patch applies with offset, runs fine, test looks good. RTBC.

lilou’s picture

lilou commented
StatusFileSize
new issue-296484.patch1.06 KB

Re-roll.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Needs work

Unfortunately this test fails for me. :( Can someone take a look?

swentel’s picture

swentel commented

weird, no problem here .. not sure what goes wrong

jonskulski’s picture

jonskulski commented
Status: Needs work » Needs review
StatusFileSize
new comment-register-296484.patch1.11 KB

@webchick: are you running out of a ~/public_html/ or similar?

Original patch assertedRaw which looked for href="/user/login...", but if you're using a RewriteBase then your urls will look something like href="/~jskulski/drupal7/user/login" and will fail the test.

This might explain what was happening, because the test was failing for me until I changed the urls to reflect my rewrite base.

Here is a rewrite of the patch that gets the links from the url function. I didn't see many examples of assertRaw with links, but this might be something to watch out for in the future.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Needs work

The last submitted patch failed testing.

emad964’s picture

emad964 commented
Component: tests » base system

There's no problem here

catch’s picture

catch
he/him
Primary language English
commented
Category: bug » task
Priority: Critical » Normal

Moving out of the critical bug queue, see #607038: Meta issue: fix gaps in code coverage

bhavesh_gloscon’s picture

bhavesh_gloscon commented

same problem when updating website Drupal 7 alpha to Drupal 7.0

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

F*cking spammers.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented

...

Status: Needs work » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.