-
-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[NFR]Whether can add an image class. #902
Comments
Does this one come from Kohana? |
Yes, I am a fan of Kohana. |
I also want to join the QR Code, though a bit inappropriate. Can you help me @sjinks ? $qr = qr_encode('test for qrcode');
if (is_resource ($qr))
qr_save ($qr, '1.png');
$qr = qr_encode ('test for qrcode');
if (is_resource ($qr))
{
header ("Content-type: image/PNG");
qr_save ($qr);
}
<?php
$qr = qr_encode ('test for qrcode', 1, QR_ECLEVEL_M, QR_MODE_KANJI, 1);
if (is_resource ($qr))
{
header ("Content-type: image/PNG");
qr_save ($qr);
} |
What do you think if we implement the Image class together? I will guide you through the process and correct the mistakes but leave the implementation to you. |
@sjinks It is my pleasure, I am very happy to get your help. |
@sjinks I have create branch https://github.com/dreamsxin/cphalcon/tree/image_class, How to start? |
@dreamsxin I will get back to you shortly — need to finish something first |
@sjinks OK. I am from now will be online 8 hours. |
@dreamsxin Please add me to GTlak — my ID is vbkolesnikov(at)gmail(dot)com |
@sjinks My ID dreamszhu(at)gmail(dot)com |
@sjinks I to the image class has not clear thinking, the implementation of the final structure such as GD, Imagick how to call, I now is to imitate the Logger class, Under the guidance of the time to help me. $image = new Phalcon\Image\Adapter\GD('5.jpg');
var_dump($image); display: object(Phalcon\Image\Adapter\GD)#1 (1) {
["_imageinfo":protected]=>
array(7) {
[0]=>
int(1600)
[1]=>
int(900)
[2]=>
int(2)
[3]=>
string(25) "width="1600" height="900""
["bits"]=>
int(8)
["channels"]=>
int(3)
["mime"]=>
string(10) "image/jpeg"
}
} |
Implemented in 1.3.0, thanks |
Functions are as follows:
The text was updated successfully, but these errors were encountered: