From: SourceForge.net <no...@so...> - 2011-11-07 13:27:06
|
Bugs item #3085775, was opened at 2010-10-12 01:20 Message generated for change (Comment added) made by kymara You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=101111&aid=3085775&group_id=1111 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: World Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Nobody/Anonymous (nobody) Assigned to: Nobody/Anonymous (nobody) Summary: Wood Not Yet Ripe Enough. Initial Comment: Suggest change of text for wood gathering as wood probably should not be classified as ripe or not like other items which are picked which are usually food related items. ---------------------------------------------------------------------- >Comment By: Katie Russell (kymara) Date: 2011-11-07 05:27 Message: For 1. here are the tracker artifacts. The history is that someone drew a 'unpicked' wood image (i.e. more fresh from the tree, not the clean image in the bag) https://sourceforge.net/tracker/index.php?func=detail&aid=1677981&group_id=1111&atid=351111 https://sourceforge.net/tracker/index.php?func=detail&aid=2688997&group_id=1111&atid=101111 ---------------------------------------------------------------------- Comment By: Katie Russell (kymara) Date: 2011-11-07 04:49 Message: 1. Wood did used to be a passive point on the ground. It was requested to change it. Then players had to 'Pick' up the wood rather than drag to bag. 2 seems fine although it's not a symmetric solution. Allow overriding of description for different levels of ripeness? In that case wood could stay using the grower. ---------------------------------------------------------------------- Comment By: RedQueen (theredqueen) Date: 2011-11-06 13:31 Message: I think we have two problems here: 1. Wood is implemented as "VegetableGrower" which is more likely to be used for items that grow in the ground (like carrots and leek). It should rather be handled as "PassiveEntityRespawnPoint" which is used for items that suddenly appear on the ground because they fell from a tree or have been produced by an animal (like apples and eggs). 2. The description of VegetableGrowers can't be overwritten by a custom message because we return static text depending on the ripeness of the grower. We have a custom description for wood in the code, but it is overwritten by the default message ("You see a planted wood seed."). My suggestions: 1. Define wood as "PassiveEntityRespawnPoint" (although it has to be dragged into the bag then, rather than just to click it) and keep the description "You see a log shaped indent in the ground.". 2. Restructure the class "VegetableGrowers" to work like this: a) Set a default description on initialization like "You see an unripe [item name]." as "seed" doesn't fit for most of the items. b) Optionally set a custom description of the grower for the unripe status like "You see a grape vine." or "You see a tomato plant.". c) When describing the item we return a text depending on the ripeness of the grower. If unripe we return the description (which is either the default or the custom description). If ripe we return "You see a ripe [item name]". What do you think? ---------------------------------------------------------------------- Comment By: Katie Russell (kymara) Date: 2010-11-28 15:19 Message: [23:19] You see a planted wood seed. likewise for the ripeness descriptions ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=101111&aid=3085775&group_id=1111 |