After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 606981 - Apply a file extensions patch to CiviCRM
Apply a file extensions patch to CiviCRM
Status: RESOLVED FIXED
Product: sysadmin
Classification: Infrastructure
Component: Other
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Sysadmins
GNOME Sysadmins
Depends on:
Blocks:
 
 
Reported: 2010-01-14 17:18 UTC by Stormy Peters
Modified: 2010-10-19 16:56 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Stormy Peters 2010-01-14 17:18:35 UTC
Can we apply this patch to CiviCRM:
 http://fisheye2.atlassian.com/changelog/~br=v3.0,author=lobo/CiviCRM/?cs=25836

It fixes an with the file extensions.

Thanks!
Comment 1 Jeff Schroeder 2010-02-10 04:46:13 UTC
The patch doesn't apply. I got it from here:
http://fisheye2.atlassian.com/rdiff/CiviCRM?csid=25836&u&N

It expects the line to be:
return CRM_Utils_String::munge( "{$basename}_{$uniqID}", '_',  240 ) . CRM_Utils_Array::value( 'extension', $info );

but instead it is currently:
return CRM_Utils_String::munge( "{$basename}_{$uniqID}", '_',  240 ) . ".{$info['extension']}";

This is line 284 in CRM/Utils/File.php. It isn't a big deal for me to manually make it what that patch does. Can you first get confirmation that is the right thing to do or would you like me to talk with CiviCRM upstream? To install it I used civicrm-3.02-drupal.tar.gz
Comment 2 Stormy Peters 2010-02-22 23:39:09 UTC
I've asked and I'll let you know when I hear back from them.
Comment 3 Stormy Peters 2010-02-23 15:56:55 UTC
Yes, that's it. I got this back from Don Lobo:


"yes that is the patch. however if u r just starting out with your civi stuff, we'd recommend you migrate to 3.1.2 (or 3.1.3 by end of week). its another step forward from a usability/feature perspective"
Comment 4 Jeff Schroeder 2010-02-23 16:05:25 UTC
*sigh* that is typical of developers. I installed the latest stable from their website when starting this but should be able to work on it some more soon.
Comment 5 Stormy Peters 2010-02-23 16:21:55 UTC
I'm ok not moving to the latest and greatest.
Comment 6 Christer Edwards 2010-10-18 18:10:42 UTC
Is this still valid?
Comment 7 Stormy Peters 2010-10-19 16:56:47 UTC
To be honest, I don't remember what I was trying to do in January ... Let's close the bug and we can reopen it if I run into the problem again. Thanks!