From 40340832e3992f46034b470aef1af6f7a3a4410f Mon Sep 17 00:00:00 2001
From: Philipp Oeser <info@graphics-engineer.com>
Date: Fri, 4 Jan 2019 14:57:45 +0100
Subject: [PATCH] OBJ Import: better handling for metallic

retrival of metallic (ab)uses OBJ material ambient term.
this can be non-zero, but in case of no reflections (OBJ 'illum' < 3)
metallic should be zero.

Fixes T60150
---
 io_scene_obj/import_obj.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/io_scene_obj/import_obj.py b/io_scene_obj/import_obj.py
index a8e088726..a065f7e1c 100644
--- a/io_scene_obj/import_obj.py
+++ b/io_scene_obj/import_obj.py
@@ -219,6 +219,9 @@ def create_materials(filepath, relpath,
             if do_reflection:
                 if "metallic" not in context_material_vars:
                     context_mat_wrap.metallic = 1.0
+            else:
+                # since we are (ab)using ambient term for metallic (which can be non-zero)
+                context_mat_wrap.metallic = 0.0
 
             if do_transparency:
                 if "ior" not in context_material_vars:
-- 
GitLab