Closed (fixed)
Project:
Panels
Version:
5.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
19 May 2008 at 15:47 UTC
Updated:
7 Jun 2008 at 05:02 UTC
Jump to comment: Most recent file
Coder module suggests changing strtolower to drupal_strtolower. Should make no big difference, except for Unicode compatibility.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | panels_strtolower_2.patch | 3.25 KB | pancho |
| panels_strtolower.patch | 582 bytes | pancho |
Comments
Comment #1
pfaocleWould also need addressing in:
Comment #2
panchoThanks Paul, you're absoutely right. Enclosed patch fixes all occurences.
Comment #3
sdboyer commentedCommitted; also found a few more and fixed em. Btw, Pancho, could you make sure to make the patches using Unified/Workspace root, rather than project root? I have to go in and manually edit them, since I also use eclipse and it picks up on the root you have (panels/...) and it doesn't correspond to my set up - I have a full drupal 5 multisite install for all my dev on d5, all of which is under svn; I then also keep certain specific modules directories under CVS control, but only the directories themselves.
Anyway, thanks!
Comment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.