It would be useful to be able to not add a Read More link if there is no regular Body field, as provided by the node module.

Comments

damienmckenna’s picture

Status: Active » Needs review
StatusFileSize
new1.94 KB

Adds a new option to the settings page to control this, which defaults to work the same way it currently does.

scribbler’s picture

Seems not to be what i hoped to be. I'm searching for something to post short messages to the frontpage which shouldn't have a 'read more'-link.
So i would write into the teaser-field and the full-text remains empty.

It would be great to have an option like:
[x] Do not add a Read More link if the full-text is empty

Could something like this be possible? After searching about this i think many user would be thankful.

aniebel’s picture

Maybe I'm missing something but I'd like it to be optional as well. Regardless of whether or not there is content in the body. I am also needed a small amount of copy on the front page and am getting a Read More link despite having nothing more to read.

robloach’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new3.73 KB
Anonymous’s picture

Rob's patch from #4 works for me!

john_b’s picture

Works for me if there is nothing in the body OR summary, just a title. But if there is text in summary, even if body is empty (I checked db that it was emtpy), the link still shows. Is that the intended behaviour?

I am confused because although Summary appears in a separate column in db, Drupal 7 treats body and teaser as part of the same field.

What would be useful is an option to remove Read More link if Summary and Body are the same. However, this is already a live issue for core http://drupal.org/node/823380 , http://drupal.org/node/1300568

A possible workaround till that issue is fixed may be to add a field called 'no read more' field to each content type which can be set when a node is created, and test for that?

john_b’s picture

Incidentally the sandbox module Read More Control gives an option to remove Read More Link when Teaser and Body are the same. It was rejected for contrib on the ground that it works around rather than fixes the core problem. It is very useful, but does not currently work when Read More Link is enabled.

polskikrol’s picture

subscribe

Anonymous’s picture

polskikrol’s picture

@BWPanda - Thanks for that, didnt even notice thats possible.

robloach’s picture

Assigned: Unassigned » robloach
Priority: Normal » Major

Ummm, didn't even realize I had commit access on this module :-P .... I'll get this in next time I'm around my development box.

robloach’s picture

Status: Reviewed & tested by the community » Fixed

Committed! http://drupalcode.org/project/read_more.git/commit/918a194

@John_B Incidentally the sandbox module Read More Control gives an option to remove Read More Link when Teaser and Body are the same. It was rejected for contrib on the ground that it works around rather than fixes the core problem. It is very useful, but does not currently work when Read More Link is enabled.

If you open up a new issue for that, we could add it tot his module.

Status: Fixed » Closed (fixed)

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

alan d.’s picture

@regards to #6 / #12

I have started the process to take over the Read More Control module. It is probably at a point where Monochrome would have been granted git access to publish if she was still actively trying.

I want to get full test coverage and to implement better default settings before doing the final push into a full project. With luck, that will be about a day away, and maybe a month for the project push (the webmaster queue is often slow). I think that there will be integration issues, but I would not worry about these until the other project is published as a full project. I create an issue / patch when the time comes.

  • Commit 918a194 on 7.x-1.x, 7.x-1.0 by RobLoach:
    Issue #1162108 by DamienMcKenna and Rob Loach: Option to hide Read More...