OFBiz
  1. OFBiz
  2. OFBIZ-3046

EditPersonTrainings has been deleted since r802308

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: SVN trunk
    • Fix Version/s: SVN trunk
    • Component/s: party
    • Labels:
      None

      Description

      EditPersonTrainings screen has been deleted from humanres/widget/PersonTrainingScreens.xml.

      To reproduce, go to https://localhost:8443/partymgr/control/viewprofile?partyId=admin and click on Training in "sub-sub"-menu

      Author: hansbak
      Date: Sat Aug 8 07:20:30 2009
      New Revision: 802308

      URL: http://svn.apache.org/viewvc?rev=802308&view=rev

      Patch file contains :
      </screen>

      • <screen name="EditPersonTrainings">
        + <screen name="TrainingCalendarWithDecorator">
        <section>

      So I don't know if EditPersonTrainings should be replaced by TrainingCalendarWithDecorator or if it should be completely deleted.
      By the way, the log of the commit is :
      Log:
      commited patch of OFBIZ-2531 with minor changes, however added initial help screens, initial commit need some more work on the permissions for admin and employee

      But there are a lot of change so may be it is a mistake ?

      1. cleantraining.patch
        5 kB
        matthieu bollot

        Issue Links

          Activity

          Hide
          Jacques Le Roux added a comment -

          Thanks Matthieu,

          Your patch is in trunk at r826272

          Show
          Jacques Le Roux added a comment - Thanks Matthieu, Your patch is in trunk at r826272
          Hide
          matthieu bollot added a comment -

          This patch delete references to EditPersonTrainings and to FindTrainings.

          Show
          matthieu bollot added a comment - This patch delete references to EditPersonTrainings and to FindTrainings.
          Hide
          Jacques Le Roux added a comment -

          Hi Matthieu,

          Actually I tried to clean this at r817443 but obvisouly I missed this one. A patch would be welcome

          Thanks

          Show
          Jacques Le Roux added a comment - Hi Matthieu, Actually I tried to clean this at r817443 but obvisouly I missed this one. A patch would be welcome Thanks

            People

            • Assignee:
              Jacques Le Roux
              Reporter:
              matthieu bollot
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development