[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

SF.net SVN: ledger-smb:[4014] branches/1.3



Revision: 4014
          http://ledger-smb.svn.sourceforge.net/ledger-smb/?rev=4014&view=rev
Author:   einhverfr
Date:     2011-11-18 00:38:18 +0000 (Fri, 18 Nov 2011)
Log Message:
-----------
Fixing bugs:
* Save new location not working on employeee screen
* Save new location not working on customer/vendor screen
* Inadequate permissions to delete locations on employee screen

Modified Paths:
--------------
    branches/1.3/LedgerSMB/ScriptLib/Company.pm
    branches/1.3/scripts/employee.pl
    branches/1.3/sql/modules/Roles.sql

Modified: branches/1.3/LedgerSMB/ScriptLib/Company.pm
===================================================================
--- branches/1.3/LedgerSMB/ScriptLib/Company.pm	2011-11-17 13:32:31 UTC (rev 4013)
+++ branches/1.3/LedgerSMB/ScriptLib/Company.pm	2011-11-18 00:38:18 UTC (rev 4014)
@@ -267,7 +267,7 @@
 
 sub save_new_location {
     my ($request) = @_;
-    delete $requet->{location_id};
+    delete $request->{location_id};
    add_location($request);
 }
 

Modified: branches/1.3/scripts/employee.pl
===================================================================
--- branches/1.3/scripts/employee.pl	2011-11-17 13:32:31 UTC (rev 4013)
+++ branches/1.3/scripts/employee.pl	2011-11-18 00:38:18 UTC (rev 4014)
@@ -76,6 +76,19 @@
 	
 }
 
+=item save_new_location 
+
+Adds a location to the company as defined in the inherited object, not
+overwriting existing locations.
+
+=cut
+
+sub save_new_location {
+    my ($request) = @_;
+    delete $request->{location_id};
+   add_location($request);
+}
+
 =item add
 
 This method creates a blank screen for entering a employee's information.

Modified: branches/1.3/sql/modules/Roles.sql
===================================================================
--- branches/1.3/sql/modules/Roles.sql	2011-11-17 13:32:31 UTC (rev 4013)
+++ branches/1.3/sql/modules/Roles.sql	2011-11-18 00:38:18 UTC (rev 4014)
@@ -169,7 +169,7 @@
 GRANT UPDATE ON person_to_company TO "lsmb_<?lsmb dbname ?>__contact_edit";
 GRANT UPDATE ON person_to_contact TO "lsmb_<?lsmb dbname ?>__contact_edit";
 GRANT UPDATE ON person_to_contact TO "lsmb_<?lsmb dbname ?>__contact_edit";
-GRANT UPDATE ON person_to_location TO "lsmb_<?lsmb dbname ?>__contact_edit";
+GRANT UPDATE, DELETE ON person_to_location TO "lsmb_<?lsmb dbname ?>__contact_edit";
 GRANT UPDATE ON eca_to_location TO "lsmb_<?lsmb dbname ?>__contact_edit";
 GRANT DELETE, INSERT  ON vendortax TO "lsmb_<?lsmb dbname ?>__contact_edit";
 GRANT DELETE, INSERT  ON entity_bank_account TO "lsmb_<?lsmb dbname ?>__contact_edit";

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.