mirror of
				https://github.com/SoftFever/OrcaSlicer.git
				synced 2025-10-30 20:21:12 -06:00 
			
		
		
		
	Fixed loading of configuraton values octoprint_host, support_material_threshold
They were incorrectly handled by the handle_legacy() function, which has been ported from the upstream Slic3r without inspecting its content.
This commit is contained in:
		
							parent
							
								
									c5dc47c647
								
							
						
					
					
						commit
						35bf4aee7d
					
				
					 1 changed files with 4 additions and 6 deletions
				
			
		|  | @ -1911,8 +1911,10 @@ void PrintConfigDef::handle_legacy(t_config_option_key &opt_key, std::string &va | |||
|         std::ostringstream oss; | ||||
|         oss << "0x0," << p.value.x << "x0," << p.value.x << "x" << p.value.y << ",0x" << p.value.y; | ||||
|         value = oss.str(); | ||||
|     } else if (opt_key == "octoprint_host" && !value.empty()) { | ||||
|         opt_key = "print_host"; | ||||
| // Maybe one day we will rename octoprint_host to print_host as it has been done in the upstream Slic3r.
 | ||||
| // Commenting this out fixes github issue #869 for now.
 | ||||
| //    } else if (opt_key == "octoprint_host" && !value.empty()) {
 | ||||
| //        opt_key = "print_host";
 | ||||
|     } else if ((opt_key == "perimeter_acceleration" && value == "25") | ||||
|         || (opt_key == "infill_acceleration" && value == "50")) { | ||||
|         /*  For historical reasons, the world's full of configs having these very low values;
 | ||||
|  | @ -1923,10 +1925,6 @@ void PrintConfigDef::handle_legacy(t_config_option_key &opt_key, std::string &va | |||
|     } else if (opt_key == "support_material_pattern" && value == "pillars") { | ||||
|         // Slic3r PE does not support the pillars. They never worked well.
 | ||||
|         value = "rectilinear"; | ||||
|     } else if (opt_key == "support_material_threshold" && value == "0") { | ||||
|         // 0 used to be automatic threshold, but we introduced percent values so let's
 | ||||
|         // transform it into the default value
 | ||||
|         value = "60%"; | ||||
|     } | ||||
|      | ||||
|     // Ignore the following obsolete configuration keys:
 | ||||
|  |  | |||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 bubnikv
						bubnikv