Remove unused public method

README.md

@@ -608,6 +608,7 @@ npm i
 4. Mask element.
 5. Load Image from URL.
 6. Fix rotate prop on Text
+7. 
 
 ### Known issues:
 1. Unable to set alpha channel for Gradient on iOS.

elements/Use.js

@@ -22,7 +22,7 @@ class Use extends Shape {
 
     render() {
         let {props} = this;
-        // 尝试匹配 "url(#pattern)"
+        // match "url(#pattern)"
         let matched = props.href.match(idExpReg);
         let href;
 

ios/Elements/RNSVGGroup.m

@@ -24,13 +24,14 @@
             svg.responsible = YES;
         }
         
-        [node mergeProperties:self mergeList:self.attributeList inherited:YES];
+        [node mergeProperties:self];
         [node renderTo:context];
         
         if ([node isKindOfClass: [RNSVGRenderable class]]) {
             RNSVGRenderable *renderable = node;
             [self concatLayoutBoundingBox:[renderable getLayoutBoundingBox]];
         }
+        
         return YES;
     }];
 }
@@ -113,12 +114,4 @@
 
 }
 
-- (void)resetProperties
-{
-    [self traverseSubviews:^(RNSVGNode *node) {
-        [node resetProperties];
-        return YES;
-    }];
-}
-
 @end

ios/Elements/RNSVGUse.m

@@ -27,9 +27,8 @@
     if (template) {
         [self beginTransparencyLayer:context];
         [self clip:context];
-        [template mergeProperties:self mergeList:self.attributeList inherited:YES];
+        [template mergeProperties:self];
         [template renderTo:context];
-        [template resetProperties];
         [self endTransparencyLayer:context];
     } else if (self.href) {
         // TODO: calling yellow box here

ios/RNSVGNode.h

@@ -68,17 +68,7 @@
  */
 - (void)saveDefinition;
 
-/**
- * just for template node to merge target node`s properties into owned properties
- */
-- (void)mergeProperties:(__kindof RNSVGNode *)target mergeList:(NSArray<NSString *> *)mergeList;
-
-- (void)mergeProperties:(__kindof RNSVGNode *)target mergeList:(NSArray<NSString *> *)mergeList inherited:(BOOL)inherited;
-
-/**
- * just for template node to reset all owned properties once after rendered.
- */
-- (void)resetProperties;
+- (void)mergeProperties:(__kindof RNSVGNode *)target;
 
 - (void)beginTransparencyLayer:(CGContextRef)context;
 

ios/RNSVGNode.m

@@ -181,12 +181,7 @@
     }
 }
 
-- (void)mergeProperties:(__kindof RNSVGNode *)target mergeList:(NSArray<NSString *> *)mergeList
-{
-    // abstract
-}
-
-- (void)mergeProperties:(__kindof RNSVGNode *)target mergeList:(NSArray<NSString *> *)mergeList inherited:(BOOL)inherited
+- (void)mergeProperties:(__kindof RNSVGNode *)target
 {
     // abstract
 }
@@ -202,11 +197,6 @@
     }
 }
 
-- (void)resetProperties
-{
-    // abstract
-}
-
 - (void)dealloc
 {
     CGPathRelease(_cachedClipPath);

ios/RNSVGRenderable.h

@@ -28,7 +28,6 @@
 @property (nonatomic, assign) RNSVGCGFloatArray strokeDasharray;
 @property (nonatomic, assign) CGFloat strokeDashoffset;
 @property (nonatomic, copy) NSArray<NSString *> *propList;
-@property (nonatomic, strong) NSArray<NSString *> *attributeList;
 
 - (void)setContextBoundingBox:(CGRect)contextBoundingBox;
 - (CGRect)getContextBoundingBox;
@@ -39,5 +38,6 @@
 - (RNSVGPercentageConverter *)getWidthConverter;
 - (RNSVGPercentageConverter *)getHeightConverter;
 - (void)setHitArea:(CGPathRef)path;
+- (NSArray<NSString *> *)getAttributeList;
 
 @end

ios/RNSVGRenderable.m

@@ -11,7 +11,8 @@
 @implementation RNSVGRenderable
 {
     NSMutableDictionary *_originProperties;
-    NSArray *_lastMergedList;
+    NSArray<NSString *> *_lastMergedList;
+    NSArray<NSString *> *_attributeList;
     RNSVGPercentageConverter *_widthConverter;
     RNSVGPercentageConverter *_heightConverter;
     CGRect _contextBoundingBox;
@@ -138,8 +139,8 @@
     if (propList == _propList) {
         return;
     }
-    _attributeList = [propList copy];
-    _propList = propList;
+    
+    _propList = _attributeList = propList;
     [self invalidate];
 }
 
@@ -347,37 +348,35 @@
     return [_heightConverter stringToFloat:string];
 }
 
-- (void)mergeProperties:(__kindof RNSVGNode *)target mergeList:(NSArray<NSString *> *)mergeList
+- (NSArray<NSString *> *)getAttributeList
 {
-    
-    [self mergeProperties:target mergeList:mergeList inherited:NO];
+    return _attributeList;
 }
 
-- (void)mergeProperties:(__kindof RNSVGNode *)target mergeList:(NSArray<NSString *> *)mergeList inherited:(BOOL)inherited
+- (void)mergeProperties:(__kindof RNSVGNode *)target
 {
-    _lastMergedList = mergeList;
+    if (_lastMergedList) {
+        [self resetProperties];
+    }
     
-    if (mergeList.count == 0) {
+    NSArray<NSString *> *targetAttributeList = [target getAttributeList];
+    
+    if (targetAttributeList.count == 0) {
         return;
     }
     
     NSMutableArray* attributeList = [self.propList mutableCopy];
-    
     _originProperties = [[NSMutableDictionary alloc] init];
     
-    for (NSString *key in mergeList) {
-        if (inherited) {
+    for (NSString *key in targetAttributeList) {
         if (![attributeList containsObject:key]) {
             [attributeList addObject:key];
             [_originProperties setValue:[self valueForKey:key] forKey:key];
             [self setValue:[target valueForKey:key] forKey:key];
         }
-        } else {
-            [_originProperties setValue:[self valueForKey:key] forKey:key];
-            [self setValue:[target valueForKey:key] forKey:key];
-        }
     }
     
+    _lastMergedList = targetAttributeList;
     _attributeList = [attributeList copy];
 }
 
@@ -386,7 +385,9 @@
     for (NSString *key in _lastMergedList) {
         [self setValue:[_originProperties valueForKey:key] forKey:key];
     }
-    _attributeList = [_propList copy];
+    
+    _lastMergedList = nil;
+    _attributeList = _propList;
 }
 
 @end

ios/RNSVGViewBox.m

@@ -160,7 +160,7 @@
     return CGAffineTransformTranslate(transform, -translateX * (_fromSymbol ? scaleX : 1), -translateY * (_fromSymbol ? scaleY : 1));
 }
 
-- (void)mergeProperties:(__kindof RNSVGNode *)target mergeList:(NSArray<NSString *> *)mergeList inherited:(BOOL)inherited
+- (void)mergeProperties:(__kindof RNSVGNode *)target
 {
     if ([target isKindOfClass:[RNSVGUse class]]) {
         RNSVGUse *use = target;

lib/attributes.js

@@ -1,9 +1,8 @@
-import _ from 'lodash';
-
-const merge = _.assign;
+const merge = Object.assign;
 
 function arrayDiffer(a, b) {
-    if (_.isNil(a) || _.isNil(b) ) {
+    /*eslint eqeqeq:0*/
+    if (a == null || b == null) {
         return true;
     }
     if (a.length !== b.length) {
@@ -22,11 +21,10 @@ function fontDiffer(a, b) {
         return false;
     }
 
-    if (a.fontSize !== b.fontSize || a.fontFamily !== b.fontFamily || a.fontStyle !== b.fontStyle || a.fontWeight !== b.fontWeight) {
-        return true;
-    }
-
-    return false;
+    return a.fontSize !== b.fontSize ||
+        a.fontFamily !== b.fontFamily ||
+        a.fontStyle !== b.fontStyle ||
+        a.fontWeight !== b.fontWeight;
 }
 
 const ViewBoxAttributes = {

lib/extract/extractOpacity.js

@@ -1,6 +1,4 @@
-import _ from 'lodash';
-
 export default function (opacity) {
-    let value = +opacity;
-    return (_.isNil(opacity) || isNaN(value)) ? 1 : value;
+    const value = +opacity;
+    return  (typeof value !== 'number' || isNaN(value)) ? 1 : value;
 }