removed unnecessary props and add constant for max filename length

components/file_attachment/filename_overlay.jsx

@@ -46,17 +46,7 @@ export default class FilenameOverlay extends React.PureComponent {
  /**
  * Optional class like for icon
  */
- iconClass: PropTypes.string,
-
- /**
- * File name
- */
- filename: PropTypes.string,
-
- /**
- * File URL
- */
- fileURL: PropTypes.string
+ iconClass: PropTypes.string
  };
 
  onAttachmentClick = (e) => {

utils/constants.jsx

@@ -540,6 +540,7 @@ export const Constants = {
  },
  MAX_DISPLAY_FILES: 5,
  MAX_UPLOAD_FILES: 5,
+ MAX_FILENAME_LENGTH: 35,
  THUMBNAIL_WIDTH: 128,
  THUMBNAIL_HEIGHT: 100,
  PROFILE_WIDTH: 128,

utils/file_utils.jsx

@@ -1,6 +1,7 @@
 // Copyright (c) 2017-present Mattermost, Inc. All Rights Reserved.
 // See License.txt for license information.
 
+import Constants from 'utils/constants.jsx';
 import * as UserAgent from 'utils/user_agent';
 
 export function canUploadFiles() {
@@ -25,8 +26,8 @@ export function canDownloadFiles() {
 
 export function trimFilename(filename) {
  let trimmedFilename = filename;
- if (filename.length > 35) {
- trimmedFilename = filename.substring(0, Math.min(35, filename.length)) + '...';
+ if (filename.length > Constants.MAX_FILENAME_LENGTH) {
+ trimmedFilename = filename.substring(0, Math.min(Constants.MAX_FILENAME_LENGTH, filename.length)) + '...';
  }
 
  return trimmedFilename;