Closed (fixed)
Project:
Excerpt
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
14 Nov 2006 at 06:36 UTC
Updated:
4 May 2007 at 06:34 UTC
Jump to comment: Most recent file
Comments
Comment #1
sillygwailoAttaching the .info file.
Comment #2
sillygwailoComment #3
sillygwailoSecond attempt at a patch. For some reason an empty teaser doesn't cause the module to use node_teaser to autogenerate a teaser as per the help text.
Comment #4
sillygwailoA third try, this time replacing $node->teaser with $form['#node']->teaser (for the edit page).
Comment #5
sillygwailoA fourth try, making it work for the new post page as well as the edit page.
Comment #6
sillygwailoComment #7
anantagati commentedWhen teaser is empty generate teaser from body.
Comment #8
peterx commentedThe changes work in RC2. I manually applied your changes from the patch file as I already had a change in Excerpt.
You remove the trim() around the teaser. I tried typing a space into teaser and it was accepted. The teaser did not display and the body was not substituted. I add back the trim.
Comment #9
amedjones commentedthis is probably a dump question, but do we apply the patch? what files do we copy it into and do we overwrite anything?
also im very curious an anxiously waiting for the official update, anyone know if the maintainers are working on it?
thanks
Comment #10
amedjones commentedok I think I figured out where to put the patch, i just overwrote everything from the excerpt.module and remove the "+" signs
but now I see the option to enable this module, nor do i see any option to configure it
any idea? what am i doing wrong?
thanks in advance
Comment #11
amedjones commentedok hopefully this is my last post for tonight
i figured out why I couldn't see the module on the site, it was because I was missing the excerpt.info, I tried to save Richard's version but i got an access denied error message
- also thank you petrex for updating the script, there was a small error on line 28, basically it was missing an extra curly bracket. I also extended the teaser to 6 rows so if you have the fckeditor installed it will replace it the text with this module.
I attached both files to this post so feel free to installed and comment on, just make sure the two files are on the same folder (i.e modules/excerpt/..
Comment #12
amedjones commentedhere is excerpt.info
Comment #13
smazsyr commentedThat link for excerpt.info seems unavialable.
Comment #14
SemperFideles commentedLooks like all the .info files don't have permissions set to be able to download. Any way to get a copy of this?
Comment #15
SemperFideles commentedRecommend anyone who has a valid excerpt.info file to post the proper code inside code tags so we can copy and paste into an .info file. I've noticed the same problem that Drupal seems to have a problem with .info attachments.
Comment #16
anantagati commentedHere is content of excerpt.info:
Comment #17
Boletus commentedI would be nice if someone could point which files are the once to use, and how.
The patch file at the top of this project issue, is that the one to use? What about the info-file? What am I supposed to do with it? I can't find any relevant information about that when I search the handbooks.
Comment #18
Boletus commentedHello again
Being a total noob when it comes to applying patches. I would much appreciate if somebody could point me in the right direction. How do I patch the 4.7 excerpt module to 5.0?
Comment #19
mjohnq3 commentedYou don't need to patch it just download excerpt.module from item #11 above and excerpt.info.txt from item #16. Rename excerpt.info.txt to excerpt.info and place both files in a folder named excerpt under .../drupal/modules/ and then goto the Drupal Modules page and activate the Excerpt Module. The Teaser text box will now appear for every node type.
Comment #20
Boletus commentedThanks a billion! :) How simple things are when you know them, and how completely annoying they are when you spend hours to try to enlighten yourself to no avail.
Works great. I am indebted. :)
Comment #21
stefano73 commentedSome code clean up. This patch is against the cvs version.
Comment #22
Christoph C. Cemper commentedwould be cool to have this in the DRUPAL-5 branch
Comment #23
mr700 commentedI'd like to see this too (subscribing).
Comment #24
ximo commentedDitto
Comment #25
SemperFideles commentedI've attached a working version of the module based stefano's patch file. I've got it working fine on Drupal 5.1.
For your excerpt.info file you can copy and paste this text:
Comment #26
robotjox commentedHi,
does anyone know how to hide the excerpt teaser from certain node types?
I have the weblink module installed and I don't need teasers for links.
thanks!
Comment #27
hesco commentedI can report that the code made available in Post #25 by Semper Fideles a few days ago, "works for me" in our environment. Thank you kindly. You were right on time.
Comment #28
redsky commentedI can verify that the code from Post #25 by Semper Fideles works for me as well in Drupal 5.1 using PostgreSQL, thanks. But like Post #26 by robotjox I can not see the "Excerpt" section under the Content Type page that in 4.7 used to let you specify whether a Teaser "Auto-generated" or "Manual excerpt".
Comment #29
jjeff commentedThis "patch" does not work correctly for me. After changing the content-type settings to "Auto-generated" teasers, a teaser field still appears on the node edit page.
Comment #30
jjeff commentedOkay, here's a patch against the current HEAD version that should work correctly. Additionally I've added a "weight" option so that you can choose where the teaser textarea shows up on the node editing page.
Comment #31
somes commentedGetting failed hunk where did you get the current head verison from
Comment #32
shipal commentedSubscribing.
Comment #33
somes commentedYep found the download of the CVS and applied the patch(file 5) seems to fix the problems described above,
Had problems getting drupal to detect it - clearing the cache on the third attempt seemed to do it.
havent come across any other issues
Comment #34
druvision commented+1 for Patch 5 - it works for me.
Unlike the new 'Node teaser' module, this module saves the teasers of previous versions.
Comment #35
pz commentedTo add to the general confusion of this thread -- here is another version of the module. No patch, complete module in txt-format, just another alternative for people too choose from.
I have only tested it on my test-site but it seems to work for me, but no guarantees.
One added feature, excerpt is shown in a (collapsed) fieldset to make it less annoying if not using it.
Still need a info file, see above or something like this
Comment #36
genx commentedCombining excerpt-5.patch, which works fine for me, with the result of a diff -N (that assumes that none existing files are blank), generates a patch file that will both
I'm uploading it as excerpt-6.patch. I hope I'm not just adding to the confusion here, and that some people might actually find it usefull :-)
P.S. This will be my very first submitted patch to Drupal, so I hope I'm doing it right :-p
Comment #37
Andrzej7 commentedStrange situation
I have the vercsion of the module made from patch (attached)
It works with Drupal 5.1 dev from 7-03-2207 and doesn't work with the version from 18-04-2007. What I mean - whatever I assign in settings (manual or automatic) it makes it automatic.
Very confusing siuation with a very, VERY important module :-(
Couldn't it be taken to core? Sorry, I'm not a coder - I can't help.
Andrzej
Comment #38
Andrzej7 commentedAdditionally:
the same with
// $Id: excerpt.module,v 1.7 2007/04/05 20:02:48 hayesr Exp $
It generates teasers (on displayed pages like example.com/blog or front page) automaticaly, even if they are not in the database!!!
The attached version from patch - makes the same things.
Comment #39
hayesr commented