alex_moz

Thank you, looks very interesting.

How about a “non-lite” version of the plugin? Is there something like that?

Thank you

Actually, “String Translation” doesn’t detect anything. Please see below on how to fix this.

Please change the “text_render” function inside of Public_Renderer.php:

public function text_render($data) {
_e(wp_kses_post(do_shortcode(str_replace('spTac', ' ', str_replace('spBac', '
', html_entity_decode($data))), $ignore_html = false)),'iheu');
}

“iheu” is just a name assigned to all strings from the plugin (this name can be used to filter all the strings inside of WPML, for easier translation).

More information here: https://wpml.org/documentation/support/enabling-text-translation-for-themes-not-compatible-with-wpml/

I hope you can include this in one of the upcoming updates.

Thank you!

• This reply was modified 1 year, 5 months ago by alex_moz.
• This reply was modified 1 year, 5 months ago by alex_moz.

Hi Biplob,

Could you please add this feature to the next release?

Thank you!

I guess it’s not, at least I don’t see the necessity at the moment.
With that option, it will make hotlinking impossible.

Adding [crypt].[ext] (actually, it can just be [crypt], without any extension) will actually put another layer of protection, if something goes wrong with the server and it doesn’t see that .htaccess, but I don’t see that as a good reason to make the change.

Thank you Jacob, I will update it now.
I also read the instructions/doc. Well written, very clear.

Regarding the direct-linking, I’m thinking to use this code in htaccess file inside of uploads/wppa and uploads/wppa/thumbs:

RewriteEngine on
RewriteCond %{HTTP_REFERER} !^https://(www\.)?domain [NC]
RewriteCond %{HTTP_REFERER} !^https://(www\.)?domain.*$ [NC]
RewriteRule \.(gif|jpg)$ – [F]

This denies direct access to the directory AND direct access to files inside of it (hot linking), unless they are fetched by my website.

Thank you again!!!

BTW, the “private” function doesn’t exist in other album plugins for wordpress (not even in nextgen, at least not in the free version of it). Event more thumbs-up for you and WPPA ??

I looked at the Console, but there are many lines there. I’ll try to figure it out there.

Definitely the Theme, as I disabled all plugins (except this one).

Thank you for your response. Here is what I get in console:

Uncaught TypeError: Cannot read property ‘split’ of undefined
jSlider.create @ jquery.slider-min.js?ver=1.5:1
jSlider.init @ jquery.slider-min.js?ver=1.5:1
b.jSlider @ jquery.slider-min.js?ver=1.5:1
b.slider @ jquery.slider-min.js?ver=1.5:1
(anonymous function) @ jquery.slider-min.js?ver=1.5:1
m.extend.each @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
m.fn.m.each @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
b.fn.slider @ jquery.slider-min.js?ver=1.5:1n._initControls
@ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:90
(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
n._create @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:90
(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
a.Widget._createWidget @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
a.(anonymous function).(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
m.extend.each @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
m.fn.m.each @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
a.fn.(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:3
1c._create @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:91
(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
a.Widget._createWidget @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
a.(anonymous function).(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
m.extend.each @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
m.fn.m.each @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
a.fn.(anonymous function) @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:31
PopMakeAdmin.initialize_color_pickers @ popup-maker-admin.min.js?ver=1.3.9:239
PopMakeAdmin.init @ popup-maker-admin.min.js?ver=1.3.9:19
(anonymous function) @ popup-maker-admin.min.js?ver=1.3.9:1048
j @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
k.fireWith @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
m.extend.ready @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2
J @ load-scripts.php?c=1&load[]=jquery-core,jquery-migrate,utils,jquery-ui-core,jquery-ui-widget,jquery…:2

I will look into them

Seems ok, I didn’t find anything wrong. I also tested it on my own server and didn’t find any more issues.
Thank you again Jacob!!!

I have the same problem.
Temporary solution that I found:
1. Edit popup-maker-admin.js
2. Comment lines 239 to 256 (everything inside of initialize_color_pickers: function () )
3. Rename that file to popup-maker-admin.min.js
4. Replace that file on the server.

Result: sliders work, but not the color-picker. That’s fine, as at least I can input colors by hand. Without that change, I wasn’t able to change most of the Theme options. Chrome console would say something about “split can’t be applied to non-object”, or something along those lines. I’m sure it’s related to wpColorPicker function.

I actually see those red messages, when I’m doing something bad (like trying to specify the album id in address bar). Normally, those messages are not there.

Wow, part 2 is very promising. I’ll help you testing/debugging it. Just tell me when!!

I wonder, when you encrypt the album ids, does the encrypted code changes? Let’s say album id=1 corresponds to id=a34d45fe (for example). Will it stay like that forever, or that code changes every login (with some random number)? Just curious.

Great, sounds good Jacob.
When do you think to make all the nice changes official as an updated release?

Hi Jacob,

I installed the update. Then used this shortcode:

[wppa type=”cover” album=”-3″]Any comment[/wppa]

but it resulted in “No albums found”. Am I doing something wrong?

I also tried, just in case:

[wppa type=”cover” album=”#owner,#me,-3″]Any comment[/wppa]

same result.

My structure:
Main_parent_album:
parent: –separate–
owner: Admin
id:5

Album1:
parent: Main_parent_album
owner: Admin
id:1

Album2:
parent: Main_parent_album
owner: user1
id:2

Album3:
parent: Main_parent_album
owner: user2
id:3

Album4:
parent: –separate–
owner: user1
id:4

Shortcode:
[wppa type=”cover” album=”#owner,#me,5″][/wppa]

Now, the album #4 (–separate– parent) never shows: good, in case the administrator forgets to change its parent to “Main_parent_album”.
Same results when #4’s parent is “none”, good.

When logged as Admin, only album #1 shows (Hmmm, could be ok too. I would expect to see all albums though).

When logged as user1: see Album2 only (don’t see Album4, see above. Nothing bad about it, in my opinion).

When logged as user2: see Album3 only

Obviously, tampering with the URL doesn’t make other albums visible, because their album_ids are encrypted ??

Next step: I will see what happens with thumbnails.

Hi Jacob,

It seems to be working great!!! WOW!!! ??

My problem: I didn’t specify the parent album previously, just #owner,#me.

Let me try all the test cases I can think of to test it (need a day or two for that), but at first try, everything works as expected! THANK YOU!!!