Image user count is not decreased when deleting a background image #40569

New Issue

Closed

opened 2014-06-10 21:27:40 +02:00 by Julien Duroure · 12 comments

Julien Duroure commented

2014-06-10 21:27:40 +02:00

Member

System Information
Linux 14.04 64bits

Blender Version
Broken: master 9016d6d
Worked: (optional)

Short description of error
Image user count is not decreased after deleting a background image

Exact steps for others to reproduce the error

Add a background image --> Open a new image file
Remove background image
Add a background image --> Link to existing blender image
Image user count is 2
Remove again background image
Add again background image --> Link to existing blender image
Image user count is now 3

I write this patch to correct this defect :
https://developer.blender.org/T40568

**System Information** Linux 14.04 64bits **Blender Version** Broken: master 9016d6d Worked: (optional) **Short description of error** Image user count is not decreased after deleting a background image **Exact steps for others to reproduce the error** * Add a background image --> Open a new image file * Remove background image * Add a background image --> Link to existing blender image * Image user count is 2 * Remove again background image * Add again background image --> Link to existing blender image * Image user count is now 3 I write this patch to correct this defect : https://developer.blender.org/T40568

Julien Duroure commented

2014-06-10 21:27:40 +02:00

Author

Member

Changed status to: 'Open'

Changed status to: 'Open'

Julien Duroure commented

2014-06-10 21:27:40 +02:00

Author

Member

Added subscriber: @JulienDuroure

Added subscriber: @JulienDuroure

Sergey Sharybin commented

2014-06-12 10:27:03 +02:00

Owner

Added subscriber: @Sergey

Added subscriber: @Sergey

Sergey Sharybin commented

2014-06-12 10:27:03 +02:00

Owner

There are actually loads of cases when usercounter goes wrong, for a long time it was only guaranteed that usercounter is correct after loading the file only. Making changes in this area so close to the release is rather realaly dangerous, so would postpone fixing it for after the release.

There are actually loads of cases when usercounter goes wrong, for a long time it was only guaranteed that usercounter is correct after loading the file only. Making changes in this area so close to the release is rather realaly dangerous, so would postpone fixing it for after the release.

Sergey Sharybin commented

2014-06-30 22:12:28 +02:00

Owner

Added subscriber: @mont29

Added subscriber: @mont29

Sergey Sharybin commented

2014-06-30 22:12:28 +02:00

Owner

@mont29, didn't you commit patch for this?

@mont29, didn't you commit patch for this?

Campbell Barton commented

2014-07-01 07:53:34 +02:00

Owner

Added subscriber: @ideasman42

Added subscriber: @ideasman42

Campbell Barton commented

2014-07-01 07:53:34 +02:00

Owner

Theres issue with patch, it can loose images now.

Theres issue with patch, it can loose images now.

Sergey Sharybin commented

2014-07-01 08:47:31 +02:00

Owner

@ideasman42, i didn't check the patch deep enough but any particular steps when images are getting lost?

@ideasman42, i didn't check the patch deep enough but any particular steps when images are getting lost?

Bastien Montagne commented

2014-07-01 09:10:44 +02:00

Owner

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'

Bastien Montagne closed this issue

2014-07-01 09:10:44 +02:00

Bastien Montagne self-assigned this 2014-07-01 09:10:44 +02:00

Bastien Montagne commented

2014-07-01 09:10:44 +02:00

Owner

Yes patch committed as 320b7a59c3

@Sergey have a look at concern raised by Campbell about this commit ;)

Yes patch committed as 320b7a59c3 @Sergey have a look at concern raised by Campbell about this commit ;)

Sergey Sharybin commented

2014-07-01 09:14:53 +02:00

Owner

Ah, oki. So it's being handled.

Ah, oki. So it's being handled.

Sign in to join this conversation.

No Label

Animation & Rigging

Asset Browser Project Overview

Automated Testing

Blender Asset Bundle

Dependency Graph

Development Management

EEVEE & Viewport

Images & Movies

Motion Tracking

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Render Pipeline

Sculpt, Paint & Texture

Video Sequencer

Virtual Reality

Blender 2.8 Project

Milestone 1: Basic, Local Asset Browser

Good First Issue

Animation & Rigging

Development Management

EEVEE & Viewport

Nodes & Physics

Pipeline, Assets & IO

Platforms, Builds & Tests

Render & Cycles

Sculpt, Paint & Texture

Needs Info from Developers

Needs Information from User

No Milestone

No project

No Assignees

4 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender#40569

No description provided.