Simplacms Не загружаются картинки в товар по https?

Question

[kycher]

Доброго дня

В cms simplacms в стандартной, даже последней версии, не загружаются изображения товара через "загрузить из интернета" по url начинающимся с https://, в базу данных пишется путь с https, фото не загружается в папку, те происходит в функции которая отвечает за добавление товара где обработка события по "add_image_url" или прерывание на передаче и обрезке url для последующего сохранения файла и записи в базу имени изображения.

Обычные файлы поправить не получилось;( (simpla/ProductAdmin.php , simpla/design/html/product.tpl)
Предполагаю что дело вообще в обработчике, скорее всего в файле api/Request.php

в simpla/design/html/product.tpl есть строки описывающие изменение на https не помогло

// Или с URL
	$('#add_image_url').click(function() {
		$("<input class='remote_image' name=images_urls[] type=text value='http://'>").appendTo('div#add_image').focus().select();
	});

в simpla/ProductAdmin.php есть строки, тоже изменение не помогло.

// Загрузка изображений из интернета
		  		    if($images = $this->request->post('images_urls'))
		  		    {
						foreach($images as $url)
						{
							if(!empty($url) && $url != 'http://')
					 			$this->products->add_image($product->id, $url);
						}
					}
					$images = $this->products->get_images(array('product_id'=>$product->id));

Не загружаются url с https на картинку. Help! Спасибо за направление на путь истинный)

Comments

[Максим Федоров]

в данном коде идет просто обработка запроса и добавление строки (сссылки) к товару в БД
а скачивание происходит в момент ресайза, когда идет запрос картинки из БД и когда ее нет локально -- запрашивается картинка по ссылке, грузится в оригиналы и ресайзится уже

затык как раз на скачивании происходит

вам в файл api/Image.php в метод resize(), а еще точнее в метод download_image()

[kycher]

Максим Федоров, да смотрел на эту api, но чет не понял где он обрезает(

api/image.php

public function download_image($filename)
	{
		// Заливаем только есть такой файл есть в базе
		$this->db->query('SELECT 1 FROM __images WHERE filename=? LIMIT 1', $filename);
		if(!$this->db->result())
			return false;
		
		// Имя оригинального файла
		$basename = explode('&', pathinfo($filename, PATHINFO_BASENAME));
		$uploaded_file = array_shift($basename);
		$base = urldecode(pathinfo($uploaded_file, PATHINFO_FILENAME));
		$ext = pathinfo($uploaded_file, PATHINFO_EXTENSION);
		
		// Если такой файл существует, нужно придумать другое название
		$new_name = urldecode($uploaded_file);
			
		while(file_exists($this->config->root_dir.$this->config->original_images_dir.$new_name))
		{
			$new_base = pathinfo($new_name, PATHINFO_FILENAME);
			if(preg_match('/_([0-9]+)$/', $new_base, $parts))
				$new_name = $base.'_'.($parts[1]+1).'.'.$ext;
			else
				$new_name = $base.'_1.'.$ext;
		}
		$this->db->query('UPDATE __images SET filename=? WHERE filename=?', $new_name, $filename);
		
		// Перед долгим копированием займем это имя
		fclose(fopen($this->config->root_dir.$this->config->original_images_dir.$new_name, 'w'));
		copy($filename, $this->config->root_dir.$this->config->original_images_dir.$new_name);
		return $new_name;
	}

Если развязывать, то смотрю он вередт в api/Config.php , где идет обработка протокола, сайт и админка тоже по https, естественно

// Протокол
		$protocol = strtolower(substr($_SERVER["SERVER_PROTOCOL"],0,5))=='https'? 'https' : 'http';
		if($_SERVER["SERVER_PORT"] == 443)
			$protocol = 'https';

		$this->vars['protocol'] = $protocol;		
		$this->vars['root_url'] = $protocol.'://'.rtrim($_SERVER['HTTP_HOST']);
		if(!empty($subdir))
			$this->vars['root_url'] .= '/'.$subdir;

[kycher]

// Загрузка изображений из интернета
		  		    if($images = $this->request->post('images_urls'))
		  		    {
						foreach($images as $url)
						{
							if(!empty($url) && $url != 'http://')
					 			$this->products->add_image($product->id, $url);
						}
					}
					$images = $this->products->get_images(array('product_id'=>$product->id));

Пробовал в условие if (!empty($url) && $url != 'http://')
добавить

if(!empty($url) && $url != 'http://') {
    $url = preg_replace("https",'http',$url);
    $this->products->add_image($product->id, $url);
}

$url != 'http://' написано для проверка пустого значения, ибо когда по умолчанию нажимаешь на эту кнопку он вставляет http:// в поле. Попытался заменить значение в url, но плохое решение, тогда не факт что по этому измененному пути будет без https отдаваться картинка.

[kycher]

Еще нашел в готовом решении подсказку.

[kycher]

// Parse the original URL
		$parse_url = parse_url($url);
		
		// Scheme and Host are always replaced
		if (isset($parts['scheme']))
			$parse_url['scheme'] = $parts['scheme'];
		if (isset($parts['host']))
			$parse_url['host'] = $parts['host'];

Вот в api/request.php есть parts['scheme'] который и есть htttp, как его поменять?

[zetwin]

В api/Image.php

...
        // Если вайл удаленный (http://), зальем его себе
        if (preg_match("~^https?://~", $source_file)) {
...

В api/Design.php

...
			if (preg_match("~^https?://~", $resized_filename_encoded)) {
            $resized_filename_encoded = rawurlencode($resized_filename_encoded);
        }
...

[kycher]

zetwin, красавчик!!! спасибо огромное!!!!! Это просто в точку!

Answer 1

[kycher]

zetwin zetwin - Правильное решение!

В api/Image.php

...
// Если вайл удаленный (http://), зальем его себе
if (preg_match("~^https?://~", $source_file)) {
...

В api/Design.php

...
if (preg_match("~^https?://~", $resized_filename_encoded)) {
$resized_filename_encoded = rawurlencode($resized_filename_encoded);
}
...

Плюс
Файл simpla\simpla_source\simpla\ProductAdmin.php

Меняем строку

if(!empty($url) && $url != 'http://' && strstr($url,'/')!==false)

на

if(!empty($url) && $url != 'http://' && $url != 'https://' && strstr($url,'/')!==false)

Answer 2

[zetwin]

В api/Image.php

...
        // Если вайл удаленный (http://), зальем его себе
        if (preg_match("~^https?://~", $source_file)) {
...

В api/Design.php

...
			if (preg_match("~^https?://~", $resized_filename_encoded)) {
            $resized_filename_encoded = rawurlencode($resized_filename_encoded);
        }
...