Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Hi Today I found this module. and tried to work with it but I am getting error while going to add new subsites. Please check the attached image.
I don't know am I missing any configuration or not. but it seems "menu_name" in "subsites" table does not allows null values. Also "menu_name" in "menu_custom" table in "Varchar(32)"
Comment | File | Size | Author |
---|---|---|---|
subsite-menu-creation error.jpg | 71.97 KB | jonline |
Comments
Comment #1
davyvdb CreditAttribution: davyvdb commentedIs the menu module enabled?
Can you give the full error message?
Does the menu_custom table exist?
Comment #2
jonline CreditAttribution: jonline commentedHi Davy
Thanks for your reply.
- Yes Menu module is enabled
- menu_custom table is exist
- I have already attached the full error message
I partially solve that issue. What I did
I have change "menu_name" field to allow "NULL" into "subsites" table.
Please check "subsites_add" method into "subsites.helpers.inc"
Here "menu_name" field is not present which is not allows "NULL" values, So its generate an users warning "Field 'menu_name' does not have a default value"
Another warning generates "Data too long for column 'menu_name' " here "menu_name" is VARCHAR(32). that means if user enter a title more than 32 character then it generate another warning.
hope this will help you.
Note: great module indeed :)
Comment #3
davyvdb CreditAttribution: davyvdb commentedThx!
Does replacing
with
fix the 2 issues?
Comment #4
jonline CreditAttribution: jonline commentedThanks for quick reply.
This will solve second issue. To solve the first issue need to change line 51 in "subsites.install"
with
Comment #5
davyvdb CreditAttribution: davyvdb commentedBut does the first issue need to be solved? There should always a menu be created. So I'd assume that if the second issue is fixed, the first one is fixed too. Could you verify this?
Comment #6
jonline CreditAttribution: jonline commentedI think this is necessary because
subsites are created here and menu_name is not present here but menu_name does not have a default value, so this will propagate a warning
Comment #7
davyvdb CreditAttribution: davyvdb commentedSeems correct! I've fixed this in CVS.
Comment #8
jonline CreditAttribution: jonline commentedThanks :)
Comment #10
marcvangendDavy, thanks for fixing this in CVS, but it took me a while to find this. Please consider to create a new release, or at least publish the nightly dev version.
Comment #11
davyvdb CreditAttribution: davyvdb commentedDone ;)
Comment #12
marcvangendSuper, thanks!