Changeset 7210


Ignore:
Timestamp:
Oct 19, 2016, 12:51:44 PM (5 years ago)
Author:
Nicklas Nordborg
Message:

References #2033: Permissions for annotating items may be incorrectly implemented

Changes in the web interface (Experimental factors in various places) so that the "edit" icon is only visible if the user has permission to modify an annotation.

Location:
trunk/www/views/experiments
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/www/views/experiments/bioassays/view_bioassay.jsp

    r6962 r7210  
    292292                        for (AnnotationSnapshot a : all)
    293293                        {
     294                          boolean annotatePermission = a.hasPermission(dc, Permission.WRITE);
    294295                          List values = a.getActualValues();
    295296                          boolean isUpToDate = a.isUpToDate();
     
    303304                            data-annotation-type="<%=at.getId() %>"
    304305                            data-annotation="<%=a.getThisAnnotationId()%>"
    305                             visible="<%=writePermission%>"
     306                            visible="<%=annotatePermission%>"
    306307                            image="<%=isUpToDate ? "edit.png" : "edit-outofsync.png"%>"
    307308                            tooltip="Modify the values of this experimental factor"
  • trunk/www/views/experiments/experiments.js

    r7001 r7210  
    155155    var numRawBioAssays = Data.int('factor-values', 'num-raw-bioassays');
    156156    var numIncomplete = 0;
    157    
    158157    for (var i = 0; i < factorValues.length; i++)
    159158    {
     
    166165     
    167166      var statusDiv = Doc.element('status.'+fv.id);
    168       var status;
     167      var status = '';
    169168      if (fv.numRawBioAssaysWithFactor >= numRawBioAssays)
    170169      {
     
    176175        status = '<img src="../../images/notify.png" alt="!">&nbsp;Inheritance not allowed';
    177176      }
    178       else
     177      else if (fv.usePermission)
    179178      {
    180179        numIncomplete++;
     
    185184        status += ' title="Select to automatically inherit annotations from parents"></td>';
    186185        status += '<label for="autoinherit'+fv.id+'">' + numMissing + ' missing</label>';
     186      }
     187      else
     188      {
     189        status = 'Read-only';
    187190      }
    188191      statusDiv.innerHTML = status;
  • trunk/www/views/experiments/rootrawbioassays/view_bioassay.jsp

    r6962 r7210  
    279279                  for (AnnotationSnapshot a : all)
    280280                  {
     281                    boolean annotatePermission = a.hasPermission(dc, Permission.WRITE);
    281282                    Unit unit = a.getActualUnit(dc);
    282283                    UnitConverter converter = null;
     
    332333                          data-annotation-type-id="<%=at.getId()%>"
    333334                          data-annotation-id="<%=a.getThisAnnotationId()%>"
    334                           tooltip="Edit this annotation" visible="<%=writePermission %>"/>
     335                          tooltip="Edit this annotation"
     336                          visible="<%=annotatePermission %>"/>
    335337                      </tbl:cell>
    336338                      <tbl:cell column="item">
  • trunk/www/views/experiments/view_experiment.jsp

    r7166 r7210  
    559559                      for (AnnotationSnapshot a : all)
    560560                      {
     561                        boolean annotatePermission = a.hasPermission(dc, Permission.WRITE);
    561562                        factorValuesCount.put(at, factorValuesCount.get(at)+1);
    562563                        List values = a.getActualValues();
     
    572573                          data-annotation-type="<%=at.getId() %>"
    573574                          data-annotation="<%=a.getThisAnnotationId()%>"
    574                           visible="<%=writePermission%>"
     575                          visible="<%=annotatePermission%>"
    575576                          image="<%=isUpToDate ? "edit.png" : "edit-outofsync.png"%>"
    576577                          tooltip="Modify the values of this experimental factor"
     
    617618          jsonAt.put("id", at.getId());
    618619          jsonAt.put("disableInheritance", at.getDisableInheritance() ? 1 : 0);
     620          jsonAt.put("usePermission", at.hasPermission(Permission.USE) ? 1 : 0);
    619621          jsonAt.put("numRawBioAssaysWithFactor", factorValuesCount.get(at));
    620622
Note: See TracChangeset for help on using the changeset viewer.