Closed (fixed)
Project:
Code per Node
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
9 Feb 2013 at 14:19 UTC
Updated:
25 Feb 2016 at 15:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
joelstein commentedYou can upload a patch; that will do. Thanks!
Comment #2
alexmoreno commentedHere is the patch, let me know if everything is ok if someone try it :-).
Comment #3
damienmckennaCorrected the issue status.
Comment #4
damienmckennaClarified the issue title.
Comment #5
damienmckennaThis needs a little work:
Comment #6
alexmoreno commentedno problem, i will make the amends.
Thank you.
Comment #7
alexmoreno commentedready. Select box added instead textform and cleaned the code searching for possible problems with Drupal coding standards.
Regarding that last point, i found a comment in the wrong place, but I am not sure if the issue in the function which you are pointing is the set_error function:
I have modified it like that:
form_set_error('cpn', t('You must select a numeric value.'));Let me know if it's right now please.
Thank you.
Comment #8
alexmoreno commentedsorry, found the issue with the Drupal coding standars in the tab. This is the right file.
Comment #10
alexmoreno commentedhow/where can i check why is this failing?
Comment #11
alexmoreno commentedI don't understand the issue, I made the patch under bluefish/Ubuntu.
I will create the park again just using vim
Comment #12
damienmckennaThe codebase had changed since you last pulled down the dev version.
I've made a few corrections to the setting, added a variable_del() line to hook_uninstall, and changed the weight values back to using JS_THEME rather than JS_LIBRARY. That said, I haven't tested it yet.
Comment #13
alexmoreno commentedthat's why it was failing my simpletest?
I will test it for you, no problem. Regarding using JS_THEME or JS_LIBRARY, I am thinking in some situations in which the administrator could want to have their js being shown before the LIBRARY (like jquery?). Could that happen? Maybe it would be a good idea to give the possibility of choose this value (JS_THEME or JS_LIBRARY)?
Thanks for your help Damien.
Comment #14
damienmckennaThere's a separate issue for adding an option to control the scope: #1294896: Option to control the scope the JS/CSS are loaded in
Comment #15
alexmoreno commentedI see it. I'd love to help, if you like my approach I can work with that idea. Assign it to me if you think I can help.
Also, i wasthinking that it would be better idea to move this global weight to the node creation itself, so instead of having the same global variable for all the nodes,the author could decide which weight give to each node individually.
Comment #16
damienmckenna@urwen: That'd be overkill, IMHO, you shouldn't need to monkey with the weight that much. If anything it might be worth expanding to have separate weights for the JS and CSS.
Comment #17
damienmckenna@urwen: How about this... First off it lets you change the CSS and JS weights separately. Secondly, it lets you manually enter an exact number for the fields, rather than only being able to adjust it slightly. This should cover for all (normal) use cases.
Comment #18
alexmoreno commentedSorry, I've been in holidays, in my home town in Spain... it's brilliant Damien, you have done a lot of good job.
I will test the path if that helps :-)
Comment #19
tusik commentedIs this patch already included in the Sept 30th 2013 dev version?
Comment #20
damienmckenna@anficr: no.
Comment #21
tusik commentedThank you.
Comment #22
alexmoreno commentedwhat do we need to include it in the next one?
Comment #23
damienmckennaComment #24
damienmckennaCommitted! Thanks for the help!
Comment #25
damienmckennaI spotted a typo after committing the change.
Comment #26
damienmckennaTypo fixed :)
Comment #27
alexmoreno commentedThanks Damien, glad to help :-)
Comment #28
damienmckennaNeeds to be backported to D6.
Comment #29
damienmckennaComment #30
alexmoreno commentedunfortunately, it cannot be done as Drupal 6 add_js and drupal_add_css does not support using a weight.
Comment #31
damienmckennaComment #32
damienmckennaThe D6 version of this module is no longer supported, so I'm closing this issue.