bugBattle for Wesnoth - Bugs: bug #19410, Image cache also keeps negative...

 
 
Show feedback again

bug #19410: Image cache also keeps negative results

Submitted by:  Ignacio R. Morelle <shadowmaster>
Submitted on:  Sun 05 Feb 2012 01:10:21 AM UTC  
 
Category: BugSeverity: 3 - Normal
Priority: 5 - NormalItem Group:  None of the others
Status: NonePrivacy: Public
Assigned to: Ali El Gariani <alink>Open/Closed: Open
Release: 1.10.0+svn, 1.11.0-svnOperating System: Most likely all

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sat 14 Apr 2012 11:43:43 PM UTC, SVN revision 53932:

Make leader_image paths in the saved games index binary path-independent (part of bug #18683)

This makes it possible to store in the save_index image paths that can
be used in the Load Game dialog even when displaying units whose
graphics are in binary paths not currently loaded. This alleviates the
effect of bug #19410 on the whole mechanism, but not completely.

When the save_index is missing, this won't work properly; we'll need to
add the leader_image attribute to the saved game config itself later.

(Browse SVN revision 53932)

Ignacio R. Morelle <shadowmaster>
Project Administrator
Sun 05 Feb 2012 01:10:21 AM UTC, original submission:

While investigating how to fix bug #18683 for units in campaigns and eras, I discovered that the image cache mechanism remembers images that could not be loaded from disk, even after loading a campaign and thus processing a [binary_path] definition that would normally enable the image to be loaded.

- To reproduce, you'll first have to break HttT; open data/campaigns/Heir_to_the_Throne/_main.cfg, and change the icon attribute under [campaign] to "units/konrad-fighter.png"
- Start Wesnoth and go to the campaign menu; the HttT entry will not have a visible icon and there'll be an image load error in stderr
- Select HttT and start the campaign; Konrad's unit baseframe will be missing from the map view and sidebar

I'd like confirmation as to whether this is working as intended, or just an oversight, since it is hindering the remainder of bug #18683.

Ignacio R. Morelle <shadowmaster>
Project Administrator

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Digest:
   bug dependencies.

 

Carbon-Copy List
  • -unavailable- added by shadowmaster (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 08 Feb 2012 08:48:39 PM UTCshadowmasterDependencies-=>bugs #18683 is dependent
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup