search

cloudinary / cloudinary_android

Android client for integrating with Cloudinary
MIT License
67 stars 59 forks source link

MediaManager changes "-" to "sub" while using public id of image in overlay method #94

Open kunall17 opened 5 years ago

kunall17 commented 5 years ago

When passing a public id products:wO_ieThMSrGaI8c-_swfzw-0000000000 in the overlay method like in the code snippet given 

Transformation x = new Transformation();
Log.d("SEEHERE", "addTagsOverlay@@@: " + x.generate());
x.overlay(alteredPublicId);
Log.d("SEEHERE", "addTagsOverlay$$: " + x.generate());

Which gives me an output of 

07-30 18:55:46.686 9350-9350/com.marsplay D/SEEHERE: addTagsOverlay@@@: 
07-30 18:55:46.687 9350-9350/com.marsplay D/SEEHERE: addTagsOverlay$$: l_products:wO_ieThMSrGaI8csub_swfzw-0000000000

Which alters the public Id of the image I've passed before. ( - changes to sub in the string)

aditimadan-Cloudinary commented 5 years ago

If the value being passed to the overlay is "products:wO_ieThMSrGaI8c-_swfzw-0000000000" then the resultant overlay value will be "l_products:wO_ieThMSrGaI8c-_swfzw-0000000000". What is the value you are expecting? Can you please elaborate the issue in detail.

kunall17 commented 5 years ago

Thanks for the reply, but the generated public id is not the same for the overlay image In the code down below 

        Transformation transformation = new Transformation();
        String publicId = "wO_ieThMSrGaI8c-_swfzw-0000000000";
        transformation.overlay(publicId);
        String result = transformation.generate();

Here I get the result string as l_wO_ieThMSrGaI8csub_swfzw-0000000000 which alters the public id of my image.

I'll highlight the difference where the publicId is changing - l_wOieThMSrGaI8`csub`swfzw-0000000000

Let me know If I am understanding things a bit wrong? We are using 1.26.0 version of cloudinary sdk

shirlymanor commented 5 years ago

@kunall17 In order to provide better feedback, can you please provide the expected result? For example: http://res.cloudinary.com/marsplay/image/upload/l_products:wO_ieThMSrGaI8c-_swfzw-0000000000,w_100/sample