{"action":"opened","issue":{"number":1068,"created_at":"2011-10-24T17:19:39Z","pull_request":{"diff_url":null,"patch_url":null,"html_url":null},"body":"In one of our apps we have placeholder names with spaces, as far as I am concern this is a valid use case.\r\n\r\nThe placeholder template tag is smart enough to understand this case, however it is missing to escape one part of the name. \r\n\r\nIn the following line\r\n\r\nhttps://github.com/divio/django-cms/blob/develop/cms/templatetags/cms_tags.py#L334\r\n\r\nIf the variable \"placeholder_name\" contains spaces it is not escaped. A simple change to \r\n\r\n\r\n        cache_key = _get_cache_key('_show_placeholder_for_page', page_lookup, lang, site_id)+'_placeholder:'+_clean_key(placeholder_name)\r\n\r\nfixed it although a better solution could be found.\r\n\r\nThe root problem here is that it's creating keys that are not compatible with memcache as it demands no spaces in the key name.","comments":0,"title":"Placeholders name with spaces break the cache ","updated_at":"2011-10-24T17:19:39Z","url":"https://api.github.com/repos/divio/django-cms/issues/1068","id":2036092,"assignee":null,"milestone":null,"closed_at":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/a57c52861cb36307efbb477b4ec3a292?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"a57c52861cb36307efbb477b4ec3a292","url":"https://api.github.com/users/elpargo","id":118551,"login":"elpargo"},"html_url":"https://github.com/divio/django-cms/issues/1068","labels":[],"state":"open"},"legacy":{"number":1068,"action":"opened","issue":2036092}}