Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 344657

Summary: [patch] Folder file encoding not reverted when m-compiler-p config removed
Product: z_Archived Reporter: Anders Hammar <anders.g.hammar>
Component: m2eAssignee: Project Inbox <m2e.core-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: igor
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Attachments:
Description Flags
Integration test showing bug
none
proposed fix
igor: iplog+
integration test none

Description Anders Hammar CLA 2011-05-04 02:32:30 EDT 
Build Identifier: 

I think there is a bug in the implementation for bug 340507, which is shown in the attached integration test.
In the case where the defined encoding is removed, the folder encoding should revert to container default (which it currently doesn't). The attached git patch for the integration tests adds a test which shows this.
I believe the simple solution is to apply the encoding to the IFolder even when
it's 'null', which means it's not defined in the Maven project.


Reproducible: Always

Steps to Reproduce:
1. Execute attached integration tests
Comment 1 Anders Hammar CLA 2011-05-04 02:33:07 EDT 
Created attachment 194666 [details]
Integration test showing bug
Comment 2 Anders Hammar CLA 2011-06-01 05:53:59 EDT 
Created attachment 197075 [details]
proposed fix

Attached fix for this bug
Comment 3 Anders Hammar CLA 2011-06-01 05:54:39 EDT 
Created attachment 197076 [details]
integration test
Comment 4 Igor Fedorenko CLA 2011-08-03 08:45:34 EDT 
Applied the patches. Thank you.
Comment 5 Denis Roy CLA 2021-04-19 13:22:53 EDT 
Moved to https://github.com/eclipse-m2e/m2e-core/issues/