Modify ↓
Opened 17 years ago
Closed 17 years ago
#5735 closed defect (fixed)
[PATCH] `split_page_names` splits two-letter words wrong
Reported by: | erose | Owned by: | Christian Boos |
---|---|---|---|
Priority: | high | Milestone: | 0.10.5 |
Component: | wiki system | Version: | devel |
Severity: | normal | Keywords: | |
Cc: | Branch: | ||
Release Notes: | |||
API Changes: | |||
Internal Changes: |
Description
If you turn on split_page_names
, "InstallPloneOnMac" renders as "Install Plone OnMac". That's wrong. This patch makes it render as "Install Plone On Mac", hopefully without breaking anything else:
-
trunk/trac/wiki/api
old new 266 266 # See http://www.w3.org/TR/REC-xml/#id, 267 267 # here adapted to exclude terminal "." and ":" characters 268 268 269 PAGE_SPLIT_RE = re.compile(r"([a-z])([A-Z] [a-z])")269 PAGE_SPLIT_RE = re.compile(r"([a-z])([A-Z])(?=[a-z])") 270 270 271 271 def format_page_name(self, page, split=False): 272 272 if split or self.split_page_names:
Attachments (0)
Change History (3)
comment:1 by , 17 years ago
Milestone: | 0.10.5 → 0.11 |
---|---|
Version: | 0.10.4 → devel |
comment:2 by , 17 years ago
Milestone: | 0.11 → 0.10.5 |
---|---|
Status: | new → assigned |
Very nice. I don't think that'll break anything.
comment:3 by , 17 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
This is a problem in 0.10.4, too. The patch is trivial to backport.