WPIntell

Source evidence

Some code for you…

Js Css Include Manager · support · 2013-07-12T14:54:00+00:00

mixedsentiment
mediumseverity
0.78relevance
5replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

5 / 25 rows with source links

20.0% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

20 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
David Anderson / Team Updraft resolved
Thanks for the plugin! Some new code for you: http://david.dw-perspective.org.uk/tmp/js-css-include-manager.1.3.0.zip I’ve added a new field – making the CSS conditional. The conditions I’ve added are (using the WP functions): – is_user_logged_in() – current_user_can(‘manage_options’) – which is the best way to detect an admin – is_front_page() That was all I needed you could add more. Even better would be multiple conditions, or allowing the user to enter their own PHP directly – but I don’t need that. I also improved some English (my mother tongue). I did not improve everything. I could not understand these sentences at all: This plug-in is a will clean the file management. You can only manage the screen. You can also only site the screen. You should also use nonces. Many thanks, David http://wordpress.org/extend/plugins/js-css-include-manager/ Thank you for your New code! and improved of the English. I took in your code of version 1.3. Then I did the test. If debug mode, Fixed bug because there was an undefined error. You should also use nonces. I’m sorry, I did not know the meaning of ‘nonces’. If you have any questions please feel free to contact me. Many thanks, gqevu6bsiz Great, thank you! Nonces protect users against XSS attacks: http://codex.wordpress.org/Glossary#Nonce http://markjaquith.wordpress.com/2006/06/02/wordpress-203-nonces/ To get more users for your plugin, you should change this description: “This plug-in is a will clean the file management. You can only manage the screen. You can also only site the screen.” I am English and I cannot understand what that means. Try something like: “This plugin allows you to include extra JavaScript or CSS files in your WordPress page. You can: * Include as many as you like. * Choose whether files are to be included on the front-end or the back-end. * Choose where to include them in the header or the footer. * Choose conditions to use for inclusion (logged-in users only, admin users only, front page only).” Best wishes, David Hi David, Thank you for your advice! And your English! My English is poor. I examined the “Nonces”. Then, I did measures. I added the wp_nonce_field() and check_admin_referer() . If you saying different, I look forward to hearing from you. If you have any questions please feel free to contact me. Many thanks, gqevu6bsiz Hi gqevu6bsiz, Your English is much better than my Japanese. 😉 I’m impressed. Looks good to me! I have been using the plugin on WordPress 3.6 beta 2, and it works fine, so you can probably update the “Tested up to:” field in readme.txt to 3.6. David Hi David, Thank you for the work reported in WordPress 3.6 beta 2. I will try to test to WordPress 3.6 beta 2 also. If there is no error, I will rewrite to 3.6 beta 2 the “Tested up to:” field of readme.txt. I will apply to the “Nonces” for my other plugins. Thank you for teaching in various ways! Thank you for reviews also! Many thanks, gqevu6bsiz

Comments

5 shown
gqevu6bsiz 2013-07-13T09:45:00+00:00

Thank you for your New code! and improved of the English. I took in your code of version 1.3. Then I did the test. If debug mode, Fixed bug because there was an undefined error. You should also use nonces. I’m sorry, I did not know the meaning of ‘nonces’. If you have any questions please feel free to contact me. Many thanks, gqevu6bsiz

David Anderson / Team Updraft 2013-07-13T14:50:00+00:00

Great, thank you! Nonces protect users against XSS attacks: http://codex.wordpress.org/Glossary#Nonce http://markjaquith.wordpress.com/2006/06/02/wordpress-203-nonces/ To get more users for your plugin, you should change this description: “This plug-in is a will clean the file management. You can only manage the screen. You can also only site the screen.” I am English and I cannot understand what that means. Try something like: “This plugin allows you to include extra JavaScript or CSS files in your WordPress page. You can: * Include as many as you like. * Choose whether files are to be included on the front-end or the back-end. * Choose where to include them in the header or the footer. * Choose conditions to use for inclusion (logged-in users only, admin users only, front page only).” Best wishes, David

gqevu6bsiz 2013-07-14T12:45:00+00:00

Hi David, Thank you for your advice! And your English! My English is poor. I examined the “Nonces”. Then, I did measures. I added the wp_nonce_field() and check_admin_referer() . If you saying different, I look forward to hearing from you. If you have any questions please feel free to contact me. Many thanks, gqevu6bsiz

David Anderson / Team Updraft 2013-07-15T08:28:00+00:00

Hi gqevu6bsiz, Your English is much better than my Japanese. 😉 I’m impressed. Looks good to me! I have been using the plugin on WordPress 3.6 beta 2, and it works fine, so you can probably update the “Tested up to:” field in readme.txt to 3.6. David

gqevu6bsiz 2013-07-15T13:31:00+00:00

Hi David, Thank you for the work reported in WordPress 3.6 beta 2. I will try to test to WordPress 3.6 beta 2 also. If there is no error, I will rewrite to 3.6 beta 2 the “Tested up to:” field of readme.txt. I will apply to the “Nonces” for my other plugins. Thank you for teaching in various ways! Thank you for reviews also! Many thanks, gqevu6bsiz