From a0534c7232fe979d50109593fb0dd4fa9f9fedda Mon Sep 17 00:00:00 2001 From: simon Date: Mon, 16 Nov 2020 19:16:06 +0200 Subject: [PATCH 01/11] move started --- .../java/net/osmand/osm/io/NetworkUtils.java | 49 ++++++++ OsmAnd/build.gradle | 2 + .../builders/cards/ImageCard.java | 1 + .../osmand/plus/osmedit/opr/OpenDBAPI.java | 107 ++++++++++++++++++ 4 files changed, 159 insertions(+) create mode 100644 OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java diff --git a/OsmAnd-java/src/main/java/net/osmand/osm/io/NetworkUtils.java b/OsmAnd-java/src/main/java/net/osmand/osm/io/NetworkUtils.java index 28a1ec3fb9..f95e3b4b45 100644 --- a/OsmAnd-java/src/main/java/net/osmand/osm/io/NetworkUtils.java +++ b/OsmAnd-java/src/main/java/net/osmand/osm/io/NetworkUtils.java @@ -56,6 +56,55 @@ public class NetworkUtils { return e.getMessage(); } } + + public static String sendPostDataRequest(String urlText, InputStream data) { + try { + log.info("POST : " + urlText); + HttpURLConnection conn = getHttpURLConnection(urlText); + conn.setDoInput(true); + conn.setDoOutput(false); + conn.setRequestMethod("POST"); + conn.setRequestProperty("Accept", "*/*"); + conn.setRequestProperty("User-Agent", "OsmAnd"); //$NON-NLS-1$ //$NON-NLS-2$ + conn.setRequestProperty("Content-Type", "multipart/form-data; boundary=" + BOUNDARY); + OutputStream ous = conn.getOutputStream(); + ous.write(("--" + BOUNDARY + "\r\n").getBytes()); + ous.write(("content-disposition: form-data; name=\"" + "file" + "\"; filename=\"" + "image1" + "\"\r\n").getBytes()); //$NON-NLS-1$ //$NON-NLS-2$ + ous.write(("Content-Type: application/octet-stream\r\n\r\n").getBytes()); //$NON-NLS-1$ + Algorithms.streamCopy(data, ous); + ous.write(("\r\n--" + BOUNDARY + "--\r\n").getBytes()); //$NON-NLS-1$ //$NON-NLS-2$ + ous.flush(); + log.info("Response code and message : " + conn.getResponseCode() + " " + conn.getResponseMessage()); + if (conn.getResponseCode() != 200) { + return null; + } + StringBuilder responseBody = new StringBuilder(); + InputStream is = conn.getInputStream(); + responseBody.setLength(0); + if (is != null) { + BufferedReader in = new BufferedReader(new InputStreamReader(is, "UTF-8")); //$NON-NLS-1$ + String s; + boolean first = true; + while ((s = in.readLine()) != null) { + if (first) { + first = false; + } else { + responseBody.append("\n"); //$NON-NLS-1$ + } + responseBody.append(s); + } + is.close(); + } + Algorithms.closeStream(is); + Algorithms.closeStream(data); + Algorithms.closeStream(ous); + return responseBody.toString(); + } catch (IOException e) { + log.error(e.getMessage(), e); + return e.getMessage(); + } + } + private static final String BOUNDARY = "CowMooCowMooCowCowCow"; //$NON-NLS-1$ public static String uploadFile(String urlText, File fileToUpload, String userNamePassword, OsmOAuthAuthorizationClient client, diff --git a/OsmAnd/build.gradle b/OsmAnd/build.gradle index c52dac72ad..91ec5744e0 100644 --- a/OsmAnd/build.gradle +++ b/OsmAnd/build.gradle @@ -526,4 +526,6 @@ dependencies { implementation 'com.jaredrummler:colorpicker:1.1.0' freehuaweiImplementation 'com.huawei.hms:iap:5.0.2.300' + + implementation "org.bouncycastle:bcpkix-jdk15on:1.56" } diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java index 26c81f03a7..d7b1d69895 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java @@ -509,6 +509,7 @@ public abstract class ImageCard extends AbstractCard { "Requesting location images...", false, false); try { if (!Algorithms.isEmpty(response)) { + //TODO extract place id JSONArray obj = new JSONObject(response).getJSONArray("objects"); JSONArray images = ((JSONObject) ((JSONObject) obj.get(0)).get("images")).getJSONArray("outdoor"); if (images.length() > 0) { diff --git a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java new file mode 100644 index 0000000000..550906a508 --- /dev/null +++ b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java @@ -0,0 +1,107 @@ +package net.osmand.plus.osmedit.opr; + +import android.net.TrafficStats; +import android.os.Build; +import com.google.gson.GsonBuilder; +import net.osmand.PlatformUtil; +import net.osmand.plus.BuildConfig; +import org.apache.commons.logging.Log; +import org.bouncycastle.jce.provider.BouncyCastleProvider; +import org.openplacereviews.opendb.SecUtils; +import org.openplacereviews.opendb.ops.OpOperation; +import org.openplacereviews.opendb.util.JsonFormatter; +import org.openplacereviews.opendb.util.exception.FailedVerificationException; + +import java.io.BufferedReader; +import java.io.DataOutputStream; +import java.io.IOException; +import java.io.InputStreamReader; +import java.net.HttpURLConnection; +import java.net.URL; +import java.security.KeyPair; +import java.security.Security; +import java.util.*; + +import static org.openplacereviews.opendb.SecUtils.*; + + +public class OpenDBAPI { + private static final Log log = PlatformUtil.getLog(SecUtils.class); + + private static final int THREAD_ID = 11200; + + public int uploadImage(String[] placeId, String privateKey, String username, String image) throws FailedVerificationException { + if (android.os.Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) { + Security.removeProvider("BC"); + Security.addProvider(new BouncyCastleProvider()); + } + KeyPair kp = SecUtils.getKeyPair(ALGO_EC, privateKey, null); + String signed = username;// + ":opr-web"; + + JsonFormatter formatter = new JsonFormatter(); + OPRImage OPRImage = new GsonBuilder().create().fromJson(image, OPRImage.class); + OpOperation opOperation = new OpOperation(); + opOperation.setType("opr.place"); + List edits = new ArrayList<>(); + Map edit = new TreeMap<>(); + List imageResponseList = new ArrayList<>(); + Map imageMap = new TreeMap<>(); + imageMap.put("cid", OPRImage.cid); + imageMap.put("hash", OPRImage.hash); + imageMap.put("extension", OPRImage.extension); + imageMap.put("type", OPRImage.type); + imageResponseList.add(imageMap); + List ids = new ArrayList<>(Arrays.asList(placeId)); + Map change = new TreeMap<>(); + Map images = new TreeMap<>(); + Map outdoor = new TreeMap<>(); + outdoor.put("outdoor", imageResponseList); + images.put("append", outdoor); + change.put("version", "increment"); + change.put("images", images); + edit.put("id", ids); + edit.put("change", change); + edit.put("current", new Object()); + edits.add(edit); + opOperation.putObjectValue(OpOperation.F_EDIT, edits); + opOperation.setSignedBy(signed); + String hash = JSON_MSG_TYPE + ":" + + SecUtils.calculateHashWithAlgo(SecUtils.HASH_SHA256, null, + formatter.opToJsonNoHash(opOperation)); + byte[] hashBytes = SecUtils.getHashBytes(hash); + String signature = signMessageWithKeyBase64(kp, hashBytes, SecUtils.SIG_ALGO_SHA1_EC, null); + opOperation.addOrSetStringValue("hash", hash); + opOperation.addOrSetStringValue("signature", signature); + TrafficStats.setThreadStatsTag(THREAD_ID); + String url = BuildConfig.OPR_BASE_URL + "api/auth/process-operation?addToQueue=true&dontSignByServer=false"; + String json = formatter.opToJson(opOperation); + System.out.println("JSON: " + json); + HttpURLConnection connection; + try { + connection = (HttpURLConnection) new URL(url).openConnection(); + connection.setRequestProperty("Content-Type", "application/json"); + connection.setConnectTimeout(10000); + connection.setRequestMethod("POST"); + connection.setDoOutput(true); + try { + DataOutputStream wr = new DataOutputStream(connection.getOutputStream()); + wr.write(json.getBytes()); + } catch (Exception e) { + e.printStackTrace(); + } + int rc = connection.getResponseCode(); + if (rc != 200) { + log.error("ERROR HAPPENED"); + BufferedReader br = new BufferedReader(new InputStreamReader(connection.getErrorStream())); + String strCurrentLine; + while ((strCurrentLine = br.readLine()) != null) { + log.error(strCurrentLine); + } + } + return rc; + } catch (IOException e) { + log.error(e); + } + return -1; + } +} \ No newline at end of file From bc6bcbcc59cae192dd986011b5e9005380672a8c Mon Sep 17 00:00:00 2001 From: simon Date: Mon, 16 Nov 2020 19:24:06 +0200 Subject: [PATCH 02/11] opr files added --- .../net/osmand/plus/osmedit/opr/OPRImage.java | 8 + .../org/openplacereviews/opendb/SecUtils.java | 217 +++++++ .../openplacereviews/opendb/ops/OpObject.java | 537 ++++++++++++++++++ .../opendb/ops/OpOperation.java | 289 ++++++++++ .../opendb/util/JsonFormatter.java | 157 +++++ .../opendb/util/JsonObjectUtils.java | 276 +++++++++ .../FailedVerificationException.java | 17 + 7 files changed, 1501 insertions(+) create mode 100644 OsmAnd/src/net/osmand/plus/osmedit/opr/OPRImage.java create mode 100644 OsmAnd/src/org/openplacereviews/opendb/SecUtils.java create mode 100644 OsmAnd/src/org/openplacereviews/opendb/ops/OpObject.java create mode 100644 OsmAnd/src/org/openplacereviews/opendb/ops/OpOperation.java create mode 100644 OsmAnd/src/org/openplacereviews/opendb/util/JsonFormatter.java create mode 100644 OsmAnd/src/org/openplacereviews/opendb/util/JsonObjectUtils.java create mode 100644 OsmAnd/src/org/openplacereviews/opendb/util/exception/FailedVerificationException.java diff --git a/OsmAnd/src/net/osmand/plus/osmedit/opr/OPRImage.java b/OsmAnd/src/net/osmand/plus/osmedit/opr/OPRImage.java new file mode 100644 index 0000000000..dfffbf9973 --- /dev/null +++ b/OsmAnd/src/net/osmand/plus/osmedit/opr/OPRImage.java @@ -0,0 +1,8 @@ +package net.osmand.plus.osmedit.opr; + +public class OPRImage { + public String type; + public String hash; + public String cid; + public String extension; +} \ No newline at end of file diff --git a/OsmAnd/src/org/openplacereviews/opendb/SecUtils.java b/OsmAnd/src/org/openplacereviews/opendb/SecUtils.java new file mode 100644 index 0000000000..183a93b9c0 --- /dev/null +++ b/OsmAnd/src/org/openplacereviews/opendb/SecUtils.java @@ -0,0 +1,217 @@ +//Revision d1a1f6e81d0716a47cbddf5754ee77fa5fc6d1d8 +package org.openplacereviews.opendb; + + +import android.util.Base64; +import org.openplacereviews.opendb.util.exception.FailedVerificationException; +import org.apache.commons.codec.DecoderException; +import org.apache.commons.codec.binary.Hex; +import org.apache.commons.codec.digest.DigestUtils; + +import java.io.ByteArrayOutputStream; +import java.io.IOException; +import java.io.UnsupportedEncodingException; +import java.security.*; +import java.security.spec.EncodedKeySpec; +import java.security.spec.InvalidKeySpecException; +import java.security.spec.PKCS8EncodedKeySpec; +import java.security.spec.X509EncodedKeySpec; +import java.util.concurrent.ThreadLocalRandom; + +//This class is a copy of SecUtils class from OpenDB project with changes for android platform +public class SecUtils { + public static final String SIG_ALGO_SHA1_EC = "SHA1withECDSA"; + public static final String SIG_ALGO_NONE_EC = "NonewithECDSA"; + + public static final String SIG_ALGO_ECDSA = "ECDSA"; + public static final String ALGO_EC = "EC"; + + public static final String DECODE_BASE64 = "base64"; + public static final String HASH_SHA256 = "sha256"; + public static final String HASH_SHA1 = "sha1"; + + public static final String JSON_MSG_TYPE = "json"; + public static final String KEY_BASE64 = DECODE_BASE64; + + + public static EncodedKeySpec decodeKey(String key) { + if (key.startsWith(KEY_BASE64 + ":")) { + key = key.substring(KEY_BASE64.length() + 1); + int s = key.indexOf(':'); + if (s == -1) { + throw new IllegalArgumentException(String.format("Key doesn't contain algorithm of hashing to verify")); + } + //should use android.util.Base64 for android platform instead of Base64.getDecoder() + return getKeySpecByFormat(key.substring(0, s), + android.util.Base64.decode(key.substring(s + 1), Base64.DEFAULT)); + } + throw new IllegalArgumentException(String.format("Key doesn't contain algorithm of hashing to verify")); + } + + public static String encodeKey(String algo, PublicKey pk) { + if (algo.equals(KEY_BASE64)) { + return SecUtils.KEY_BASE64 + ":" + pk.getFormat() + ":" + encodeBase64(pk.getEncoded()); + } + throw new UnsupportedOperationException("Algorithm is not supported: " + algo); + } + + public static String encodeKey(String algo, PrivateKey pk) { + if (algo.equals(KEY_BASE64)) { + return SecUtils.KEY_BASE64 + ":" + pk.getFormat() + ":" + encodeBase64(pk.getEncoded()); + } + throw new UnsupportedOperationException("Algorithm is not supported: " + algo); + } + + public static EncodedKeySpec getKeySpecByFormat(String format, byte[] data) { + switch (format) { + case "PKCS#8": + return new PKCS8EncodedKeySpec(data); + case "X.509": + return new X509EncodedKeySpec(data); + } + throw new IllegalArgumentException(format); + } + + public static String encodeBase64(byte[] data) { + //should use android.util.Base64 for android platform instead of Base64.getDecoder() + return new String(android.util.Base64.decode(data, android.util.Base64.DEFAULT)); + } + + public static boolean validateKeyPair(String algo, PrivateKey privateKey, PublicKey publicKey) + throws FailedVerificationException { + if (!algo.equals(ALGO_EC)) { + throw new FailedVerificationException("Algorithm is not supported: " + algo); + } + // create a challenge + byte[] challenge = new byte[512]; + ThreadLocalRandom.current().nextBytes(challenge); + + try { + // sign using the private key + Signature sig = Signature.getInstance(SIG_ALGO_SHA1_EC); + sig.initSign(privateKey); + sig.update(challenge); + byte[] signature = sig.sign(); + + // verify signature using the public key + sig.initVerify(publicKey); + sig.update(challenge); + + boolean keyPairMatches = sig.verify(signature); + return keyPairMatches; + } catch (InvalidKeyException e) { + throw new FailedVerificationException(e); + } catch (NoSuchAlgorithmException e) { + throw new FailedVerificationException(e); + } catch (SignatureException e) { + throw new FailedVerificationException(e); + } + } + + public static KeyPair getKeyPair(String algo, String prKey, String pbKey) throws FailedVerificationException { + try { + KeyFactory keyFactory = KeyFactory.getInstance(algo); + PublicKey pb = null; + PrivateKey pr = null; + if (pbKey != null) { + pb = keyFactory.generatePublic(decodeKey(pbKey)); + } + if (prKey != null) { + pr = keyFactory.generatePrivate(decodeKey(prKey)); + } + return new KeyPair(pb, pr); + } catch (NoSuchAlgorithmException e) { + throw new FailedVerificationException(e); + } catch (InvalidKeySpecException e) { + throw new FailedVerificationException(e); + } + } + + public static String signMessageWithKeyBase64(KeyPair keyPair, byte[] msg, String signAlgo, ByteArrayOutputStream out) { + byte[] sigBytes; + try { + sigBytes = signMessageWithKey(keyPair, msg, signAlgo); + } catch (FailedVerificationException e) { + throw new IllegalStateException("Cannot get bytes"); + } + if (out != null) { + try { + out.write(sigBytes); + } catch (IOException e) { + throw new IllegalStateException(e); + } + } + String signature = Base64.encodeToString(sigBytes, Base64.DEFAULT).replace("\n", ""); + return signAlgo + ":" + DECODE_BASE64 + ":" + signature; + } + + public static byte[] signMessageWithKey(KeyPair keyPair, byte[] msg, String signAlgo) throws FailedVerificationException { + try { + //use BouncyCastle on android platform in order to achieve consistency between platforms + Signature sig = Signature.getInstance(getInternalSigAlgo(signAlgo), "BC"); + sig.initSign(keyPair.getPrivate()); + sig.update(msg); + byte[] signatureBytes = sig.sign(); + return signatureBytes; + } catch (NoSuchAlgorithmException e) { + throw new FailedVerificationException(e); + } catch (InvalidKeyException e) { + throw new FailedVerificationException(e); + } catch (SignatureException e) { + throw new FailedVerificationException(e); + } catch (NoSuchProviderException e) { + throw new RuntimeException(e); + } + } + + private static String getInternalSigAlgo(String sigAlgo) { + return sigAlgo.equals(SIG_ALGO_ECDSA) ? SIG_ALGO_NONE_EC : sigAlgo; + } + + public static byte[] calculateHash(String algo, byte[] b1, byte[] b2) { + byte[] m = mergeTwoArrays(b1, b2); + if (algo.equals(HASH_SHA256)) { + return DigestUtils.sha256(m); + } else if (algo.equals(HASH_SHA1)) { + return DigestUtils.sha1(m); + } + throw new UnsupportedOperationException(); + } + + public static byte[] mergeTwoArrays(byte[] b1, byte[] b2) { + byte[] m = b1 == null ? b2 : b1; + if (b2 != null && b1 != null) { + m = new byte[b1.length + b2.length]; + System.arraycopy(b1, 0, m, 0, b1.length); + System.arraycopy(b2, 0, m, b1.length, b2.length); + } + return m; + } + + public static String calculateHashWithAlgo(String algo, String salt, String msg) { + try { + //use Hex.encodeHex for android platform instead of Hex.encodeHexString + char[] hex = Hex.encodeHex(calculateHash(algo, salt == null ? null : salt.getBytes("UTF-8"), + msg == null ? null : msg.getBytes("UTF-8"))); + + return algo + ":" + new String(hex); + } catch (UnsupportedEncodingException e) { + throw new IllegalStateException(e); + } + } + + public static byte[] getHashBytes(String msg) { + if (msg == null || msg.length() == 0) { + // special case for empty hash + return new byte[0]; + } + int i = msg.lastIndexOf(':'); + String s = i >= 0 ? msg.substring(i + 1) : msg; + try { + return Hex.decodeHex(s.toCharArray()); + } catch (DecoderException e) { + throw new IllegalArgumentException(e); + } + } + +} diff --git a/OsmAnd/src/org/openplacereviews/opendb/ops/OpObject.java b/OsmAnd/src/org/openplacereviews/opendb/ops/OpObject.java new file mode 100644 index 0000000000..2cc8acc839 --- /dev/null +++ b/OsmAnd/src/org/openplacereviews/opendb/ops/OpObject.java @@ -0,0 +1,537 @@ +//Revision d1a1f6e81d0716a47cbddf5754ee77fa5fc6d1d8 +package org.openplacereviews.opendb.ops; + +import com.google.gson.*; +import org.openplacereviews.opendb.util.JsonObjectUtils; +// OSMAND ANDROID CHANGE BEGIN: +// removed unused imports +// OSMAND ANDROID CHANGE END + +import java.lang.reflect.Type; +import java.text.ParseException; +import java.text.SimpleDateFormat; +import java.util.*; +import java.util.Map.Entry; +import java.util.concurrent.ConcurrentHashMap; + +public class OpObject { + + public static final String F_NAME = "name"; + public static final String F_ID = "id"; + public static final String F_COMMENT = "comment"; + public static final String TYPE_OP = "sys.op"; + public static final String TYPE_BLOCK = "sys.block"; + public static final String DATE_FORMAT = "yyyy-MM-dd'T'HH:mm:ss.SSSZ"; + // transient info about validation timing etc + public static final String F_EVAL = "eval"; + public static final String F_VALIDATION = "validation"; + public static final String F_TIMESTAMP_ADDED = "timestamp"; + public static final String F_PARENT_TYPE = "parentType"; + public static final String F_PARENT_HASH = "parentHash"; + public static final String F_CHANGE = "change"; + public static final String F_CURRENT = "current"; + // voting + public static final String F_OP = "op"; + public static final String F_STATE = "state"; + public static final String F_OPEN = "open"; + public static final String F_FINAL = "final"; + public static final String F_VOTE = "vote"; + public static final String F_VOTES = "votes"; + public static final String F_SUBMITTED_OP_HASH = "submittedOpHash"; + public static final String F_USER = "user"; + + public static final OpObject NULL = new OpObject(true); + + public static SimpleDateFormat dateFormat = new SimpleDateFormat(DATE_FORMAT); + static { + dateFormat.setTimeZone(TimeZone.getTimeZone("UTC")); + } + + protected Map fields = new TreeMap<>(); + protected transient Map cacheFields; + protected boolean isImmutable; + + protected transient String parentType; + protected transient String parentHash; + protected transient boolean deleted; + + + public OpObject() {} + + public OpObject(boolean deleted) { + this.deleted = deleted; + } + + public OpObject(OpObject cp) { + this(cp, false); + } + + public OpObject(OpObject cp, boolean copyCacheFields) { + createOpObjectCopy(cp, copyCacheFields); + } + + @SuppressWarnings("unchecked") + private OpObject createOpObjectCopy(OpObject opObject, boolean copyCacheFields) { + this.parentType = opObject.parentType; + this.parentHash = opObject.parentHash; + this.deleted = opObject.deleted; + this.fields = (Map) copyingObjects(opObject.fields, copyCacheFields); + if (opObject.cacheFields != null && copyCacheFields) { + this.cacheFields = (Map) copyingObjects(opObject.cacheFields, copyCacheFields); + } + this.isImmutable = false; + + return this; + } + + public boolean isDeleted() { + return deleted; + } + + @SuppressWarnings("unchecked") + private Object copyingObjects(Object object, boolean copyCacheFields) { + if (object instanceof Number) { + return (Number) object; + } else if (object instanceof String) { + return (String) object; + } else if (object instanceof Boolean) { + return (Boolean) object; + } else if (object instanceof List) { + List copy = new ArrayList<>(); + List list = (List) object; + for (Object o : list) { + copy.add(copyingObjects(o, copyCacheFields)); + } + return copy; + } else if (object instanceof Map) { + Map copy = new LinkedHashMap<>(); + Map map = (Map) object; + for (Object o : map.keySet()) { + copy.put(o, copyingObjects(map.get(o), copyCacheFields)); + } + return copy; + } + // OSMAND ANDROID CHANGE BEGIN: + // removed instanceOf OpExprEvaluator + // OSMAND ANDROID CHANGE END: + else if (object instanceof OpObject) { + return new OpObject((OpObject) object); + } else { + throw new UnsupportedOperationException("Type of object is not supported"); + } + } + + public void setParentOp(OpOperation op) { + setParentOp(op.type, op.getRawHash()); + } + + public void setParentOp(String parentType, String parentHash) { + this.parentType = parentType; + this.parentHash = parentHash; + } + + public String getParentHash() { + return parentHash; + } + + public String getParentType() { + return parentType; + } + + public List getId() { + return getStringList(F_ID); + } + + public void setId(String id) { + addOrSetStringValue(F_ID, id); + } + + public boolean isImmutable() { + return isImmutable; + } + + public OpObject makeImmutable() { + isImmutable = true; + return this; + } + + public Object getFieldByExpr(String field) { + if (field.contains(".") || field.contains("[") || field.contains("]")) { + return JsonObjectUtils.getField(this.fields, generateFieldSequence(field)); + } + + return fields.get(field); + } + + + /** + * generateFieldSequence("a") - [a] + * generateFieldSequence("a.b") - [a, b] + * generateFieldSequence("a.b.c.de") - [a, b, c, de] + * generateFieldSequence("a.bwerq.c") - [a, bwerq, c] + * generateFieldSequence("a.bwerq...c") - [a, bwerq, c] + * generateFieldSequence("a.bwereq..c..") - [a, bwerq, c] + * generateFieldSequence("a.{b}") - [a, b] + * generateFieldSequence("a.{b.c.de}") - [a, b.c.de] + * generateFieldSequence("a.{b.c.de}") - [a, b.c.de] + * generateFieldSequence("a.{b{}}") - [a, b{}] + * generateFieldSequence("a.{b{}d.q}") - [a, b{}d.q] + */ + private static List generateFieldSequence(String field) { + int STATE_OPEN_BRACE = 1; + int STATE_OPEN = 0; + int state = STATE_OPEN; + int start = 0; + List l = new ArrayList(); + for(int i = 0; i < field.length(); i++) { + boolean split = false; + if (i == field.length() - 1) { + if (state == STATE_OPEN_BRACE) { + if(field.charAt(i) == '}') { + split = true; + } else { + throw new IllegalArgumentException("Illegal field expression: " + field); + } + } else { + if(field.charAt(i) != '.') { + i++; + } + split = true; + } + } else { + if (field.charAt(i) == '.' && state == STATE_OPEN) { + split = true; + } else if (field.charAt(i) == '}' && field.charAt(i + 1) == '.' && state == STATE_OPEN_BRACE) { + split = true; + } else if (field.charAt(i) == '{' && state == STATE_OPEN) { + if(start != i) { + throw new IllegalArgumentException("Illegal field expression (wrap {} is necessary): " + field); + } + state = STATE_OPEN_BRACE; + start = i + 1; + } + } + if(split) { + if (i != start) { + l.add(field.substring(start, i)); + } + start = i + 1; + state = STATE_OPEN; + } + } + return l; + } + + public void setFieldByExpr(String field, Object object) { + if (field.contains(".") || field.contains("[") || field.contains("]")) { + List fieldSequence = generateFieldSequence(field); + if (object == null) { + JsonObjectUtils.deleteField(this.fields, fieldSequence); + } else { + JsonObjectUtils.setField(this.fields, fieldSequence, object); + } + } else if (object == null) { + fields.remove(field); + } else { + fields.put(field, object); + } + } + + + public Object getCacheObject(String f) { + if(cacheFields == null) { + return null; + } + return cacheFields.get(f); + } + + public void putCacheObject(String f, Object o) { + if (isImmutable()) { + if (cacheFields == null) { + cacheFields = new ConcurrentHashMap(); + } + cacheFields.put(f, o); + } + } + + public void setId(String id, String id2) { + List list = new ArrayList(); + list.add(id); + list.add(id2); + putObjectValue(F_ID, list); + } + + public String getName() { + return getStringValue(F_NAME); + } + + public String getComment() { + return getStringValue(F_COMMENT); + } + + public Map getRawOtherFields() { + return fields; + } + + @SuppressWarnings("unchecked") + public Map getStringMap(String field) { + return (Map) fields.get(field); + } + + @SuppressWarnings("unchecked") + public Map> getMapStringList(String field) { + return (Map>) fields.get(field); + } + + @SuppressWarnings("unchecked") + public List> getListStringMap(String field) { + return (List>) fields.get(field); + } + + @SuppressWarnings("unchecked") + public List> getListStringObjMap(String field) { + return (List>) fields.get(field); + } + + @SuppressWarnings("unchecked") + public Map getStringObjMap(String field) { + return (Map) fields.get(field); + } + + @SuppressWarnings("unchecked") + public T getField(T def, String... fields) { + Map p = this.fields; + for(int i = 0; i < fields.length - 1 ; i++) { + p = (Map) p.get(fields[i]); + if(p == null) { + return def; + } + } + T res = (T) p.get(fields[fields.length - 1]); + if(res == null) { + return def; + } + return res; + } + + @SuppressWarnings("unchecked") + public Map, Object> getStringListObjMap(String field) { + return (Map, Object>) fields.get(field); + } + + + public long getDate(String field) { + String date = getStringValue(field); + // OSMAND ANDROID CHANGE BEGIN: + // removed check OUtils.isEmpty(date) + // OSMAND ANDROID CHANGE END + try { + return dateFormat.parse(date).getTime(); + } catch (ParseException e) { + return 0; + } + } + + + public void setDate(String field, long time) { + putStringValue(field, dateFormat.format(new Date(time))); + } + + public Number getNumberValue(String field) { + return (Number) fields.get(field); + } + + public int getIntValue(String key, int def) { + Number o = getNumberValue(key); + return o == null ? def : o.intValue(); + } + + public long getLongValue(String key, long def) { + Number o = getNumberValue(key); + return o == null ? def : o.longValue(); + } + + public String getStringValue(String field) { + Object o = fields.get(field); + if (o instanceof String || o == null) { + return (String) o; + } + return o.toString(); + } + + @SuppressWarnings("unchecked") + public List getStringList(String field) { + // cast to list if it is single value + Object o = fields.get(field); + if(o == null || o.toString().isEmpty()) { + return Collections.emptyList(); + } + if(o instanceof String) { + return Collections.singletonList(o.toString()); + } + return (List) o; + } + + public Object getObjectValue(String field) { + return fields.get(field); + } + + public void putStringValue(String key, String value) { + checkNotImmutable(); + if(value == null) { + fields.remove(key); + } else { + fields.put(key, value); + } + } + + /** + * Operates as a single value if cardinality is less than 1 + * or as a list of values if it stores > 1 value + * @param key + * @param value + */ + @SuppressWarnings("unchecked") + public void addOrSetStringValue(String key, String value) { + checkNotImmutable(); + Object o = fields.get(key); + if(o == null) { + fields.put(key, value); + } else if(o instanceof List) { + ((List) o).add(value); + } else { + List list = new ArrayList(); + list.add(o.toString()); + list.add(value); + fields.put(key, list); + } + } + + @SuppressWarnings("unchecked") + public Map getChangedEditFields() { + return (Map) fields.get(F_CHANGE); + } + + @SuppressWarnings("unchecked") + public Map getCurrentEditFields() { + return (Map) fields.get(F_CURRENT); + } + + public void putObjectValue(String key, Object value) { + checkNotImmutable(); + if(value == null) { + fields.remove(key); + } else { + fields.put(key, value); + } + } + + public void checkNotImmutable() { + if(isImmutable) { + throw new IllegalStateException("Object is immutable"); + } + + } + + public void checkImmutable() { + if(!isImmutable) { + throw new IllegalStateException("Object is mutable"); + } + } + + public Object remove(String key) { + checkNotImmutable(); + return fields.remove(key); + } + + public Map getMixedFieldsAndCacheMap() { + TreeMap mp = new TreeMap<>(fields); + if(cacheFields != null || parentType != null || parentHash != null) { + TreeMap eval = new TreeMap(); + + if(parentType != null) { + eval.put(F_PARENT_TYPE, parentType); + } + if(parentHash != null) { + eval.put(F_PARENT_HASH, parentHash); + } + if (cacheFields != null) { + Iterator> it = cacheFields.entrySet().iterator(); + while (it.hasNext()) { + Entry e = it.next(); + Object v = e.getValue(); + if (v instanceof Map || v instanceof String || v instanceof Number) { + eval.put(e.getKey(), v); + } + } + } + if(eval.size() > 0) { + mp.put(F_EVAL, eval); + } + } + return mp; + } + + @Override + public int hashCode() { + final int prime = 31; + int result = 1; + result = prime * result + ((fields == null) ? 0 : fields.hashCode()); + return result; + } + + @Override + public String toString() { + return getClass().getSimpleName() + "[" + fields + "]"; + } + + @Override + public boolean equals(Object obj) { + if (this == obj) + return true; + if (obj == null) + return false; + if (getClass() != obj.getClass()) + return false; + OpObject other = (OpObject) obj; + if (fields == null) { + if (other.fields != null) + return false; + } else if (!fields.equals(other.fields)) + return false; + return true; + } + + public static class OpObjectAdapter implements JsonDeserializer, + JsonSerializer { + + private boolean fullOutput; + + public OpObjectAdapter(boolean fullOutput) { + this.fullOutput = fullOutput; + } + + @Override + public OpObject deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) + throws JsonParseException { + OpObject bn = new OpObject(); + bn.fields = context.deserialize(json, TreeMap.class); + // remove cache + bn.fields.remove(F_EVAL); + return bn; + } + + @Override + public JsonElement serialize(OpObject src, Type typeOfSrc, JsonSerializationContext context) { + return context.serialize(fullOutput ? src.getMixedFieldsAndCacheMap() : src.fields); + } + + + } + + + + + + +} diff --git a/OsmAnd/src/org/openplacereviews/opendb/ops/OpOperation.java b/OsmAnd/src/org/openplacereviews/opendb/ops/OpOperation.java new file mode 100644 index 0000000000..27f2b5fb91 --- /dev/null +++ b/OsmAnd/src/org/openplacereviews/opendb/ops/OpOperation.java @@ -0,0 +1,289 @@ +//Revision d1a1f6e81d0716a47cbddf5754ee77fa5fc6d1d8 +package org.openplacereviews.opendb.ops; + +import com.google.gson.*; + +import java.lang.reflect.Type; +import java.util.*; +// OSMAND ANDROID CHANGE BEGIN: +// removed dependency OUtils +// OSMAND ANDROID CHANGE END: +public class OpOperation extends OpObject { + + public static final String F_TYPE = "type"; + public static final String F_SIGNED_BY = "signed_by"; + public static final String F_HASH = "hash"; + + public static final String F_SIGNATURE = "signature"; + + public static final String F_REF = "ref"; + public static final String F_CREATE = "create"; + public static final String F_DELETE = "delete"; + public static final String F_EDIT = "edit"; + + public static final String F_NAME = "name"; + public static final String F_COMMENT = "comment"; + + private List createdObjects = new LinkedList(); + private List editedObjects = new LinkedList(); + protected String type; + + public OpOperation() { + } + + public OpOperation(OpOperation cp, boolean copyCacheFields) { + super(cp, copyCacheFields); + this.type = cp.type; + for(OpObject o : cp.createdObjects) { + this.createdObjects.add(new OpObject(o, copyCacheFields)); + } + for(OpObject o : cp.editedObjects) { + this.editedObjects.add(new OpObject(o, copyCacheFields)); + } + } + + public String getOperationType() { + return type; + } + + public void setType(String name) { + checkNotImmutable(); + type = name; + updateObjectsRef(); + } + + protected void updateObjectsRef() { + for(OpObject o : createdObjects) { + o.setParentOp(this); + } + for(OpObject o : editedObjects) { + o.setParentOp(this); + } + } + + public String getType() { + return type; + } + + public OpOperation makeImmutable() { + isImmutable = true; + for(OpObject o : createdObjects) { + o.makeImmutable(); + } + for(OpObject o : editedObjects) { + o.makeImmutable(); + } + return this; + } + + public void setSignedBy(String value) { + putStringValue(F_SIGNED_BY, value); + } + + public void addOtherSignedBy(String value) { + super.addOrSetStringValue(F_SIGNED_BY, value); + } + + public List getSignedBy() { + return getStringList(F_SIGNED_BY); + } + + public String getHash() { + return getStringValue(F_HASH); + } + + public String getRawHash() { + String rw = getStringValue(F_HASH); + // drop algorithm and everything else + if(rw != null) { + rw = rw.substring(rw.lastIndexOf(':') + 1); + } + return rw; + } + + public List getSignatureList() { + return getStringList(F_SIGNATURE); + } + + public Map> getRef() { + return getMapStringList(F_REF); + } + + @SuppressWarnings("unchecked") + public List> getDeleted() { + List> l = (List>) fields.get(F_DELETE); + if(l == null) { + return Collections.emptyList(); + } + return l; + } + + public boolean hasDeleted() { + return getDeleted().size() > 0; + } + + public void addDeleted(List id) { + if(!fields.containsKey(F_DELETE)) { + ArrayList> lst = new ArrayList<>(); + lst.add(id); + putObjectValue(F_DELETE, lst); + } else { + getDeleted().add(id); + } + } + + public List getCreated() { + return createdObjects; + } + + public void addCreated(OpObject o) { + checkNotImmutable(); + createdObjects.add(o); + if(type != null) { + o.setParentOp(this); + } + } + + public boolean hasCreated() { + return createdObjects.size() > 0; + } + + public void addEdited(OpObject o) { + checkNotImmutable(); + editedObjects.add(o); + if (type != null) { + o.setParentOp(this); + } + } + + public List getEdited() { + return editedObjects; + } + + public boolean hasEdited() { + return editedObjects.size() > 0; + } + + + public String getName() { + return getStringValue(F_NAME); + } + + public String getComment() { + return getStringValue(F_COMMENT); + } + + @Override + public int hashCode() { + final int prime = 31; + int result = super.hashCode(); + result = prime * result + ((createdObjects == null) ? 0 : createdObjects.hashCode()); + result = prime * result + ((editedObjects == null) ? 0 : editedObjects.hashCode()); + result = prime * result + ((type == null) ? 0 : type.hashCode()); + return result; + } + + @Override + public boolean equals(Object obj) { + if (this == obj) + return true; + if (!super.equals(obj)) + return false; + if (getClass() != obj.getClass()) + return false; + OpOperation other = (OpOperation) obj; + if (createdObjects == null) { + if (other.createdObjects != null) + return false; + } else if (!createdObjects.equals(other.createdObjects)) + return false; + if (editedObjects == null) { + if (other.editedObjects != null) + return false; + } else if (!editedObjects.equals(other.editedObjects)) + return false; + if (type == null) { + if (other.type != null) + return false; + } else if (!type.equals(other.type)) + return false; + return true; + } + + // OSMAND ANDROID CHANGE BEGIN: + // removed unused classes and methods + // public static OpObjectDiffBuilder createDiffOperation(OpObject o) + // private static Object diffSet(Object vl) + // public static class OpObjectDiffBuilder{} + // OSMAND ANDROID CHANGE END + + public static class OpOperationBeanAdapter implements JsonDeserializer, + JsonSerializer { + + // plain serialization to calculate hash + private boolean excludeHashAndSignature; + private boolean fullOutput; + + public OpOperationBeanAdapter(boolean fullOutput, boolean excludeHashAndSignature) { + this.excludeHashAndSignature = excludeHashAndSignature; + this.fullOutput = fullOutput; + } + + public OpOperationBeanAdapter(boolean fullOutput) { + this.fullOutput = fullOutput; + this.excludeHashAndSignature = false; + } + + @Override + public OpOperation deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) + throws JsonParseException { + JsonObject jsonObj = json.getAsJsonObject(); + OpOperation op = new OpOperation(); + JsonElement tp = jsonObj.remove(F_TYPE); + if(tp != null) { + String opType = tp.getAsString(); + op.type = opType; + } else { + op.type = ""; + } + JsonElement createdObjs = jsonObj.remove(F_CREATE); + if(createdObjs != null) { + JsonArray ar = createdObjs.getAsJsonArray(); + for(int i = 0; i < ar.size(); i++) { + op.addCreated((OpObject) context.deserialize(ar.get(i), OpObject.class)); + } + } + + JsonElement editedObjs = jsonObj.remove(F_EDIT); + if (editedObjs != null) { + for (JsonElement editElem : editedObjs.getAsJsonArray()) { + op.addEdited((OpObject) context.deserialize(editElem, OpObject.class)); + } + } + + jsonObj.remove(F_EVAL); + op.fields = context.deserialize(jsonObj, TreeMap.class); + return op; + } + + @Override + public JsonElement serialize(OpOperation src, Type typeOfSrc, JsonSerializationContext context) { + TreeMap tm = new TreeMap<>(fullOutput ? src.getMixedFieldsAndCacheMap() : src.fields); + if(excludeHashAndSignature) { + tm.remove(F_SIGNATURE); + tm.remove(F_HASH); + } + tm.put(F_TYPE, src.type); + + if (src.hasEdited()) { + tm.put(F_EDIT, context.serialize(src.editedObjects)); + } + + if(src.hasCreated()) { + tm.put(F_CREATE, context.serialize(src.createdObjects)); + } + return context.serialize(tm); + } + } + +} \ No newline at end of file diff --git a/OsmAnd/src/org/openplacereviews/opendb/util/JsonFormatter.java b/OsmAnd/src/org/openplacereviews/opendb/util/JsonFormatter.java new file mode 100644 index 0000000000..6603f4c3ab --- /dev/null +++ b/OsmAnd/src/org/openplacereviews/opendb/util/JsonFormatter.java @@ -0,0 +1,157 @@ +//Revision d1a1f6e81d0716a47cbddf5754ee77fa5fc6d1d8 +package org.openplacereviews.opendb.util; + +import com.google.gson.*; +// OSMAND ANDROID CHANGE BEGIN: +// removed dependency org.openplacereviews.opendb.ops.OpBlock; +// OSMAND ANDROID CHANGE END +import org.openplacereviews.opendb.ops.OpObject; +import org.openplacereviews.opendb.ops.OpOperation; +// OSMAND ANDROID CHANGE BEGIN: +// removed dependency org.springframework.stereotype.Component; +// OSMAND ANDROID CHANGE END + +import java.io.Reader; +import java.lang.reflect.Type; +import java.util.*; + +// OSMAND ANDROID CHANGE BEGIN: +// removed annotation @Component +// OSMAND ANDROID CHANGE END +public class JsonFormatter { + + private Gson gson; + + private Gson gsonOperationHash; + + private Gson gsonFullOutput; + + public JsonFormatter() { + GsonBuilder builder = new GsonBuilder(); + builder.disableHtmlEscaping(); + builder.registerTypeAdapter(OpOperation.class, new OpOperation.OpOperationBeanAdapter(false)); + builder.registerTypeAdapter(OpObject.class, new OpObject.OpObjectAdapter(false)); + // OSMAND ANDROID CHANGE BEGIN: + // removed OpBlock.class TypeAdapter + // OSMAND ANDROID CHANGE END + builder.registerTypeAdapter(TreeMap.class, new MapDeserializerDoubleAsIntFix()); + gson = builder.create(); + + builder = new GsonBuilder(); + builder.disableHtmlEscaping(); + builder.registerTypeAdapter(OpOperation.class, new OpOperation.OpOperationBeanAdapter(false, true)); + builder.registerTypeAdapter(OpObject.class, new OpObject.OpObjectAdapter(false)); + // OSMAND ANDROID CHANGE BEGIN: + // removed OpBlock.class TypeAdapter + // OSMAND ANDROID CHANGE END + builder.registerTypeAdapter(TreeMap.class, new MapDeserializerDoubleAsIntFix()); + gsonOperationHash = builder.create(); + + builder = new GsonBuilder(); + builder.disableHtmlEscaping(); + builder.registerTypeAdapter(OpOperation.class, new OpOperation.OpOperationBeanAdapter(true)); + builder.registerTypeAdapter(OpObject.class, new OpObject.OpObjectAdapter(true)); + builder.registerTypeAdapter(TreeMap.class, new MapDeserializerDoubleAsIntFix()); + gsonFullOutput = builder.create(); + + + } + + public static class MapDeserializerDoubleAsIntFix implements JsonDeserializer> { + + @Override @SuppressWarnings("unchecked") + public TreeMap deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context) throws JsonParseException { + return (TreeMap) read(json); + } + + public Object read(JsonElement in) { + + if(in.isJsonArray()){ + List list = new ArrayList(); + JsonArray arr = in.getAsJsonArray(); + for (JsonElement anArr : arr) { + list.add(read(anArr)); + } + return list; + }else if(in.isJsonObject()){ + Map map = new TreeMap(); + JsonObject obj = in.getAsJsonObject(); + Set> entitySet = obj.entrySet(); + for(Map.Entry entry: entitySet){ + map.put(entry.getKey(), read(entry.getValue())); + } + return map; + }else if(in.isJsonPrimitive()){ + JsonPrimitive prim = in.getAsJsonPrimitive(); + if(prim.isBoolean()){ + return prim.getAsBoolean(); + }else if(prim.isString()){ + return prim.getAsString(); + }else if(prim.isNumber()){ + Number num = prim.getAsNumber(); + // here you can handle double int/long values + // and return any type you want + // this solution will transform 3.0 float to long values + if(Math.ceil(num.doubleValue()) == num.longValue() && (!num.toString().contains(".") || num.toString().split("\\.")[1].length() <= 1)) + return num.longValue(); + else { + return num.doubleValue(); + } + } + } + return null; + } + } + +// operations to parse / format related + public OpOperation parseOperation(String opJson) { + return gson.fromJson(opJson, OpOperation.class); + } + + public OpObject parseObject(String opJson) { + return gson.fromJson(opJson, OpObject.class); + } + + // OSMAND ANDROID CHANGE BEGIN: + // removed unused methods + // public OpBlock parseBlock(String opJson) + // public String toJson(OpBlock bl) + // OSMAND ANDROID CHANGE END + + public JsonElement toJsonElement(Object o) { + return gson.toJsonTree(o); + } + + @SuppressWarnings("unchecked") + public TreeMap fromJsonToTreeMap(String json) { + return gson.fromJson(json, TreeMap.class); + } + + + public T fromJson(Reader json, Class classOfT) throws JsonSyntaxException { + return gson.fromJson(json, classOfT); + } + + public T fromJson(Reader json, Type typeOfT) throws JsonSyntaxException { + return gson.fromJson(json, typeOfT); + } + + public String fullObjectToJson(Object o) { + return gsonFullOutput.toJson(o); + } + + + public String opToJsonNoHash(OpOperation op) { + return gsonOperationHash.toJson(op); + } + + public String opToJson(OpOperation op) { + return gson.toJson(op); + } + + public String objToJson(OpObject op) { + return gson.toJson(op); + } + + +} diff --git a/OsmAnd/src/org/openplacereviews/opendb/util/JsonObjectUtils.java b/OsmAnd/src/org/openplacereviews/opendb/util/JsonObjectUtils.java new file mode 100644 index 0000000000..f372394c8b --- /dev/null +++ b/OsmAnd/src/org/openplacereviews/opendb/util/JsonObjectUtils.java @@ -0,0 +1,276 @@ +//Revision d1a1f6e81d0716a47cbddf5754ee77fa5fc6d1d8 +package org.openplacereviews.opendb.util; + +import java.util.*; + +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; + +/** + * Class uses for work with Json Object represent as Map. + */ +public class JsonObjectUtils { + + + private static final int GET_OPERATION = 0; + private static final int SET_OPERATION = 1; + private static final int DELETE_OPERATION = 2; + protected static final Log LOGGER = LogFactory.getLog(JsonObjectUtils.class); + + private static class OperationAccess { + private final int operation; + private final Object value; + + private OperationAccess(int op, Object v) { + this.operation = op; + this.value = v; + } + + } + + /** + * Retrieve value from jsonMap by field sequence. + * @param jsonMap source json object deserialized in map + * @param fieldSequence Sequence to field value. + * Example: person.car.number have to be ["person", "car[2]", "number"] + * @return Field value + */ + public static Object getField(Map jsonMap, String[] fieldSequence) { + return accessField(jsonMap, fieldSequence, new OperationAccess(GET_OPERATION, null)); + } + + /** + * Set value to json field (path to field presented as sequence of string) + * + * @param jsonMap source json object deserialized in map + * @param fieldSequence Sequence to field value. + * * Example: person.car.number have to be ["person", "car[2]", "number"] + * @param field field value + * @return + */ + public static Object setField(Map jsonMap, List fieldSequence, Object field) { + return setField(jsonMap, fieldSequence.toArray(new String[fieldSequence.size()]), field); + } + + /** + * Set value to json field (path to field presented as sequence of string) + * + * @param jsonObject source json object deserialized in map + * @param fieldSequence Sequence to field value. + * * Example: person.car.number have to be ["person", "car[2]", "number"] + * @param field field value + * @return + */ + public static Object setField(Map jsonObject, String[] fieldSequence, Object field) { + return accessField(jsonObject, fieldSequence, new OperationAccess(SET_OPERATION, field)); + } + + + /** + * Retrieve value from jsonMap by field sequence. + * + * @param jsonObject source json object deserialized in map + * @param fieldSequence Sequence to field value. + * Example: person.car.number have to be ["person", "car[2]", "number"] + * @return Field value + */ + public static Object getField(Map jsonObject, List fieldSequence) { + return getField(jsonObject, fieldSequence.toArray(new String[fieldSequence.size()])); + } + + /** + * Delete field value from json Map (field path presented as sequence of string) + * + * @param jsonMap source json object deserialized in map + * @param fieldSequence Sequence to field value. + * Example: person.car.number have to be ["person", "car[2]", "number"] + * @return + */ + public static Object deleteField(Map jsonMap, List fieldSequence) { + return accessField(jsonMap, fieldSequence.toArray(new String[fieldSequence.size()]), new OperationAccess(DELETE_OPERATION, null)); + } + + + @SuppressWarnings("unchecked") + private static Object accessField(Map jsonObject, String[] fieldSequence, OperationAccess op) { + if (fieldSequence == null || fieldSequence.length == 0) { + throw new IllegalArgumentException("Field sequence is empty. Set value to root not possible."); + } + String fieldName = null; + Map jsonObjLocal = jsonObject; + List jsonListLocal = null; + int indexToAccess = -1; + for(int i = 0; i < fieldSequence.length; i++) { + boolean last = i == fieldSequence.length - 1; + fieldName = fieldSequence[i]; + int indOpArray = -1; + for(int ic = 0; ic < fieldName.length(); ) { + if(ic > 0 && (fieldName.charAt(ic) == '[' || fieldName.charAt(ic) == ']') && + fieldName.charAt(ic - 1) == '\\') { + // replace '\[' with '[' + fieldName = fieldName.substring(0, ic - 1) + fieldName.substring(ic); + } else if(fieldName.charAt(ic) == '[') { + indOpArray = ic; + break; + } else { + ic++; + } + } + jsonListLocal = null; // reset + if(indOpArray == -1) { + if(!last) { + Map fieldAccess = (Map) jsonObjLocal.get(fieldName); + if(fieldAccess == null) { + if(op.operation == GET_OPERATION) { + // don't modify during get operation + return null; + } + Map newJsonMap = new TreeMap<>(); + jsonObjLocal.put(fieldName, newJsonMap); + jsonObjLocal = newJsonMap; + } else { + jsonObjLocal = fieldAccess; + } + } + } else { + String arrayFieldName = fieldName.substring(0, indOpArray); + if(arrayFieldName.contains("]")) { + throw new IllegalArgumentException(String.format("Illegal field array modifier %s", fieldSequence[i])); + } + jsonListLocal = (List) jsonObjLocal.get(arrayFieldName); + if (jsonListLocal == null) { + if (op.operation == GET_OPERATION) { + // don't modify during get operation + return null; + } + jsonListLocal = new ArrayList(); + jsonObjLocal.put(arrayFieldName, jsonListLocal); + } + while (indOpArray != -1) { + fieldName = fieldName.substring(indOpArray + 1); + int indClArray = fieldName.indexOf("]"); + if (indClArray == -1) { + throw new IllegalArgumentException(String.format("Illegal field array modifier %s", fieldSequence[i])); + } + if(indClArray == fieldName.length() - 1) { + indOpArray = -1; + } else if(fieldName.charAt(indClArray + 1) == '[') { + indOpArray = indClArray + 1; + } else { + throw new IllegalArgumentException(String.format("Illegal field array modifier %s", fieldSequence[i])); + } + int index = Integer.parseInt(fieldName.substring(0, indClArray)); + if (last && indOpArray == -1) { + indexToAccess = index; + } else { + Object obj = null; + if (index < jsonListLocal.size() && index >= 0) { + obj = jsonListLocal.get(index); + } else if (op.operation == SET_OPERATION && (index == -1 || index == jsonListLocal.size())) { + index = jsonListLocal.size(); + jsonListLocal.add(null); + } else { + throw new IllegalArgumentException( + String.format("Illegal access to array at position %d", index)); + } + + if (obj == null) { + if (op.operation == GET_OPERATION) { + // don't modify during get operation + return null; + } + if (indOpArray == -1) { + obj = new TreeMap<>(); + } else { + obj = new ArrayList(); + } + jsonListLocal.set(index, obj); + } + if(indOpArray != -1) { + jsonListLocal = (List) obj; + } else { + jsonObjLocal = (Map) obj; + jsonListLocal = null; + } + } + } + + } + } + if(jsonListLocal != null) { + return accessListField(op, jsonListLocal, indexToAccess); + } else { + return accessObjField(op, jsonObjLocal, fieldName); + } + } + + private static Object accessObjField(OperationAccess op, Map jsonObjLocal, String fieldName) { + Object prevValue; + if (op.operation == DELETE_OPERATION) { + prevValue = jsonObjLocal.remove(fieldName); + } else if (op.operation == SET_OPERATION) { + prevValue = jsonObjLocal.put(fieldName, op.value); + } else { + prevValue = jsonObjLocal.get(fieldName); + } + return prevValue; + } + + private static Object accessListField(OperationAccess op, List jsonListLocal, int indexToAccess) { + Object prevValue; + int lastIndex = indexToAccess; + if (op.operation == DELETE_OPERATION) { + if (lastIndex >= jsonListLocal.size() || lastIndex < 0) { + prevValue = null; + } else { + prevValue = jsonListLocal.remove(lastIndex); + } + } else if (op.operation == SET_OPERATION) { + if (lastIndex == jsonListLocal.size() || lastIndex == -1) { + prevValue = null; + jsonListLocal.add(op.value); + } else if (lastIndex >= jsonListLocal.size() || lastIndex < 0) { + throw new IllegalArgumentException(String.format("Illegal access to %d position in array with size %d", + lastIndex, jsonListLocal.size())); + } else { + prevValue = jsonListLocal.set(lastIndex, op.value); + } + } else { + if (lastIndex >= jsonListLocal.size() || lastIndex < 0) { + prevValue = null; + } else { + prevValue = jsonListLocal.get(lastIndex); + } + } + return prevValue; + } + + @SuppressWarnings("unchecked") + public static List getIndexObjectByField(Object obj, List field, List res) { + if(obj == null) { + return res; + } + if(field.size() == 0) { + if(res == null) { + res = new ArrayList(); + } + res.add(obj); + return res; + } + if (obj instanceof Map) { + String fieldFirst = field.get(0); + Object value = ((Map) obj).get(fieldFirst); + return getIndexObjectByField(value, field.subList(1, field.size()), res); + } else if(obj instanceof Collection) { + for(Object o : ((Collection)obj)) { + res = getIndexObjectByField(o, field, res); + } + } else { + // we need extract but there no field + LOGGER.warn(String.format("Can't access field %s for object %s", field, obj)); + } + return res; + } + + +} diff --git a/OsmAnd/src/org/openplacereviews/opendb/util/exception/FailedVerificationException.java b/OsmAnd/src/org/openplacereviews/opendb/util/exception/FailedVerificationException.java new file mode 100644 index 0000000000..8826742e42 --- /dev/null +++ b/OsmAnd/src/org/openplacereviews/opendb/util/exception/FailedVerificationException.java @@ -0,0 +1,17 @@ +//Revision d1a1f6e81d0716a47cbddf5754ee77fa5fc6d1d8 +package org.openplacereviews.opendb.util.exception; + +public class FailedVerificationException extends Exception { + + private static final long serialVersionUID = -4936205097177668159L; + + + public FailedVerificationException(Exception e) { + super(e); + } + + + public FailedVerificationException(String msg) { + super(msg); + } +} \ No newline at end of file From 8ea9ba2925de6e5433cae3e4122b13a0d1b8578e Mon Sep 17 00:00:00 2001 From: simon Date: Tue, 17 Nov 2020 15:42:21 +0200 Subject: [PATCH 03/11] add photo hidden --- OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java index 45c4864a2a..73301158c5 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java @@ -336,6 +336,8 @@ public class MenuBuilder { b.setTypeface(null, Typeface.BOLD); b.setText(context.getResources().getString(R.string.shared_string_add_photo)); b.setBackgroundResource(R.drawable.btn_border_light); + //TODO feature under development + b.setVisibility(View.GONE); b.setTextColor(ContextCompat.getColor(context, R.color.preference_category_title)); return b; } From 2bde2da819f7b6fd4543a58ca186c8b584780def Mon Sep 17 00:00:00 2001 From: simon Date: Tue, 17 Nov 2020 17:07:16 +0200 Subject: [PATCH 04/11] adding image id code --- .../builders/cards/ImageCard.java | 75 ++++++++++++------- 1 file changed, 50 insertions(+), 25 deletions(-) diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java index d7b1d69895..1108692934 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java @@ -21,6 +21,7 @@ import net.osmand.Location; import net.osmand.PlatformUtil; import net.osmand.data.Amenity; import net.osmand.data.LatLon; +import net.osmand.plus.BuildConfig; import net.osmand.plus.OsmandApplication; import net.osmand.plus.R; import net.osmand.plus.Version; @@ -39,13 +40,7 @@ import org.json.JSONObject; import java.text.DateFormat; import java.text.ParseException; import java.text.SimpleDateFormat; -import java.util.ArrayList; -import java.util.Collections; -import java.util.Date; -import java.util.LinkedHashMap; -import java.util.List; -import java.util.Locale; -import java.util.Map; +import java.util.*; public abstract class ImageCard extends AbstractCard { @@ -408,6 +403,28 @@ public abstract class ImageCard extends AbstractCard { } } + private static String[] getIdFromResponse(String response) { + try { + JSONArray obj = new JSONObject(response).getJSONArray("objects"); + JSONArray images = (JSONArray) ((JSONObject) obj.get(0)).get("id"); + return toStringArray(images); + } catch (JSONException e) { + e.printStackTrace(); + } + return new String[0]; + } + + private static String[] toStringArray(JSONArray array) { + if (array == null) + return null; + + String[] arr = new String[array.length()]; + for (int i = 0; i < arr.length; i++) { + arr[i] = array.optString(i); + } + return arr; + } + public static class GetImageCardsTask extends AsyncTask> { private MapActivity mapActivity; @@ -425,7 +442,7 @@ public abstract class ImageCard extends AbstractCard { } public GetImageCardsTask(@NonNull MapActivity mapActivity, LatLon latLon, - @Nullable Map params, GetImageCardsListener listener) { + @Nullable Map params, GetImageCardsListener listener) { this.mapActivity = mapActivity; this.app = mapActivity.getMyApplication(); this.latLon = latLon; @@ -441,7 +458,15 @@ public abstract class ImageCard extends AbstractCard { if (o instanceof Amenity) { Amenity am = (Amenity) o; long amenityId = am.getId() >> 1; - getPicturesForPlace(result, amenityId); + String url = BuildConfig.OPR_BASE_URL + "api/objects-by-index?type=opr.place&index=osmid&key=" + amenityId; + String response = AndroidNetworkUtils.sendRequest(app, url, Collections.emptyMap(), + "Requesting location images...", false, false); + if (response != null) { + getPicturesForPlace(result, response); + String[] id = getIdFromResponse(response); + //TODO perform something with image + //listener.onOPRPlaceIdAcquired(id); + } } try { final Map pms = new LinkedHashMap<>(); @@ -503,27 +528,27 @@ public abstract class ImageCard extends AbstractCard { return result; } - private void getPicturesForPlace(List result, long l) { - String url = "https://test.openplacereviews.org/api/objects-by-index?type=opr.place&index=osmid&limit=1&key=" + l; - String response = AndroidNetworkUtils.sendRequest(app, url, Collections.emptyMap(), - "Requesting location images...", false, false); + private void getPicturesForPlace(List result, String response) { try { if (!Algorithms.isEmpty(response)) { - //TODO extract place id JSONArray obj = new JSONObject(response).getJSONArray("objects"); - JSONArray images = ((JSONObject) ((JSONObject) obj.get(0)).get("images")).getJSONArray("outdoor"); - if (images.length() > 0) { - for (int i = 0; i < images.length(); i++) { - try { - JSONObject imageObject = (JSONObject) images.get(i); - if (imageObject != JSONObject.NULL) { - ImageCard imageCard = ImageCard.createCardOpr(mapActivity, imageObject); - if (imageCard != null) { - result.add(imageCard); + JSONObject imagesWrapper = ((JSONObject) ((JSONObject) obj.get(0)).get("images")); + Iterator it = imagesWrapper.keys(); + while (it.hasNext()) { + JSONArray images = imagesWrapper.getJSONArray(it.next()); + if (images.length() > 0) { + for (int i = 0; i < images.length(); i++) { + try { + JSONObject imageObject = (JSONObject) images.get(i); + if (imageObject != JSONObject.NULL) { + ImageCard imageCard = ImageCard.createCardOpr(mapActivity, imageObject); + if (imageCard != null) { + result.add(imageCard); + } } + } catch (JSONException e) { + LOG.error(e); } - } catch (JSONException e) { - LOG.error(e); } } } From 14b0ab427a055f4699b963585ef63d0473b0ff52 Mon Sep 17 00:00:00 2001 From: simon Date: Tue, 17 Nov 2020 18:17:53 +0200 Subject: [PATCH 05/11] add check email functions --- .../osmand/plus/osmedit/opr/OpenDBAPI.java | 25 ++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) diff --git a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java index 550906a508..5214ec2df5 100644 --- a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java +++ b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java @@ -3,7 +3,9 @@ package net.osmand.plus.osmedit.opr; import android.net.TrafficStats; import android.os.Build; import com.google.gson.GsonBuilder; +import net.osmand.AndroidNetworkUtils; import net.osmand.PlatformUtil; +import net.osmand.osm.io.NetworkUtils; import net.osmand.plus.BuildConfig; import org.apache.commons.logging.Log; import org.bouncycastle.jce.provider.BouncyCastleProvider; @@ -27,9 +29,30 @@ import static org.openplacereviews.opendb.SecUtils.*; public class OpenDBAPI { private static final Log log = PlatformUtil.getLog(SecUtils.class); - + private static final String checkLoginEndpoint = "api/auth/user-check-loginkey?"; + private static final String LOGIN_SUCCESS_MESSAGE = "success"; private static final int THREAD_ID = 11200; + /* + * method for check if user is loggined in blockchain + * params + * - username: blockchain username in format "openplacereviews:test_1" + * - privatekey: "base64:PKCS#8:actualKey" + * + * Do not call on mainThread + */ + public boolean checkPrivateKeyValid(String username, String privateKey){ + String url = BuildConfig.OPR_BASE_URL + checkLoginEndpoint + + "name=" + + username + + "&" + + "privateKey=" + + privateKey; + StringBuilder response = new StringBuilder(); + return (NetworkUtils.sendGetRequest(url,null,response) == null) && + response.toString().contains(LOGIN_SUCCESS_MESSAGE); + } + public int uploadImage(String[] placeId, String privateKey, String username, String image) throws FailedVerificationException { if (android.os.Build.VERSION.SDK_INT >= Build.VERSION_CODES.P) { Security.removeProvider("BC"); From 5cef32531b88e54f249a072b64cfa7adf02766c6 Mon Sep 17 00:00:00 2001 From: simon Date: Wed, 18 Nov 2020 18:21:25 +0200 Subject: [PATCH 06/11] encode url bug check login fixed --- .../osmand/plus/osmedit/opr/OpenDBAPI.java | 28 +++++++++++-------- 1 file changed, 16 insertions(+), 12 deletions(-) diff --git a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java index 5214ec2df5..0d77c49d25 100644 --- a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java +++ b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java @@ -14,12 +14,10 @@ import org.openplacereviews.opendb.ops.OpOperation; import org.openplacereviews.opendb.util.JsonFormatter; import org.openplacereviews.opendb.util.exception.FailedVerificationException; -import java.io.BufferedReader; -import java.io.DataOutputStream; -import java.io.IOException; -import java.io.InputStreamReader; +import java.io.*; import java.net.HttpURLConnection; import java.net.URL; +import java.net.URLEncoder; import java.security.KeyPair; import java.security.Security; import java.util.*; @@ -38,16 +36,22 @@ public class OpenDBAPI { * params * - username: blockchain username in format "openplacereviews:test_1" * - privatekey: "base64:PKCS#8:actualKey" - * + * Need to encode key * Do not call on mainThread */ - public boolean checkPrivateKeyValid(String username, String privateKey){ - String url = BuildConfig.OPR_BASE_URL + checkLoginEndpoint + - "name=" + - username + - "&" + - "privateKey=" + - privateKey; + public boolean checkPrivateKeyValid(String username, String privateKey) throws UnsupportedEncodingException { + String url = null; + try { + url = BuildConfig.OPR_BASE_URL + checkLoginEndpoint + + "name=" + + username + + "&" + + "privateKey=" + + //need to encode the key + URLEncoder.encode(privateKey, "UTF-8"); + } catch (UnsupportedEncodingException e) { + throw e; + } StringBuilder response = new StringBuilder(); return (NetworkUtils.sendGetRequest(url,null,response) == null) && response.toString().contains(LOGIN_SUCCESS_MESSAGE); From c307a0e4fe21cc16d8f9f303de9db1726a5048d1 Mon Sep 17 00:00:00 2001 From: Simon Yakymovych Date: Fri, 20 Nov 2020 12:26:23 +0200 Subject: [PATCH 07/11] photo upload feature implemented --- OsmAnd/res/values/strings.xml | 2 + .../plus/mapcontextmenu/MenuBuilder.java | 127 +++++++++++++++++- .../builders/cards/ImageCard.java | 5 +- .../osmand/plus/osmedit/opr/OpenDBAPI.java | 8 +- 4 files changed, 135 insertions(+), 7 deletions(-) diff --git a/OsmAnd/res/values/strings.xml b/OsmAnd/res/values/strings.xml index 83862c5287..9029f205e2 100644 --- a/OsmAnd/res/values/strings.xml +++ b/OsmAnd/res/values/strings.xml @@ -11,6 +11,8 @@ Thx - Hardy --> + Select picture + Cannot upload image, please, try again later Motorboat Kayak Search history diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java index 73301158c5..30b7f05cd8 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java @@ -6,6 +6,8 @@ import android.content.Context; import android.content.DialogInterface; import android.content.Intent; import android.content.res.ColorStateList; +import android.graphics.Bitmap; +import android.graphics.BitmapFactory; import android.graphics.Color; import android.graphics.PorterDuff; import android.graphics.Typeface; @@ -33,11 +35,14 @@ import androidx.appcompat.view.ContextThemeWrapper; import androidx.core.content.ContextCompat; import androidx.core.graphics.drawable.DrawableCompat; import net.osmand.AndroidUtils; +import net.osmand.PlatformUtil; import net.osmand.data.Amenity; import net.osmand.data.LatLon; import net.osmand.data.PointDescription; import net.osmand.data.QuadRect; +import net.osmand.osm.io.NetworkUtils; import net.osmand.plus.*; +import net.osmand.plus.activities.ActivityResultListener; import net.osmand.plus.activities.MapActivity; import net.osmand.plus.helpers.FontCache; import net.osmand.plus.mapcontextmenu.builders.cards.AbstractCard; @@ -46,7 +51,9 @@ import net.osmand.plus.mapcontextmenu.builders.cards.ImageCard; import net.osmand.plus.mapcontextmenu.builders.cards.ImageCard.GetImageCardsTask; import net.osmand.plus.mapcontextmenu.builders.cards.NoImagesCard; import net.osmand.plus.mapcontextmenu.controllers.TransportStopController; +import net.osmand.plus.openplacereviews.OPRWebviewActivity; import net.osmand.plus.openplacereviews.OprStartFragment; +import net.osmand.plus.osmedit.opr.OpenDBAPI; import net.osmand.plus.poi.PoiUIFilter; import net.osmand.plus.render.RenderingIcons; import net.osmand.plus.transport.TransportStopRoute; @@ -57,6 +64,14 @@ import net.osmand.plus.widgets.tools.ClickableSpanTouchListener; import net.osmand.util.Algorithms; import net.osmand.util.MapUtils; +import org.apache.commons.logging.Log; +import org.openplacereviews.opendb.util.exception.FailedVerificationException; + +import java.io.BufferedInputStream; +import java.io.ByteArrayInputStream; +import java.io.ByteArrayOutputStream; +import java.io.InputStream; +import java.text.MessageFormat; import java.util.*; import static net.osmand.plus.mapcontextmenu.builders.cards.ImageCard.GetImageCardsTask.GetImageCardsListener; @@ -92,6 +107,12 @@ public class MenuBuilder { private String preferredMapAppLang; private boolean transliterateNames; + private static final int PICK_IMAGE = 1231; + private static final Log LOG = PlatformUtil.getLog(MenuBuilder.class); + private View view; + private OpenDBAPI openDBAPI = new OpenDBAPI(); + private String[] placeId = new String[0]; + public interface CollapseExpandListener { void onCollapseExpand(boolean collapsed); } @@ -336,8 +357,103 @@ public class MenuBuilder { b.setTypeface(null, Typeface.BOLD); b.setText(context.getResources().getString(R.string.shared_string_add_photo)); b.setBackgroundResource(R.drawable.btn_border_light); + b.setOnClickListener(new OnClickListener() { + @Override + public void onClick(final View view) { + mapActivity.registerActivityResultListener(new ActivityResultListener(PICK_IMAGE, + new ActivityResultListener.OnActivityResultListener() { + @Override + public void onResult(int resultCode, Intent resultData) { + InputStream inputStream = null; + try { + inputStream = mapActivity.getContentResolver().openInputStream(resultData.getData()); + } catch (Exception e) { + LOG.error(e); + } + handleSelectedImage(view, inputStream); + } + })); + final String privateKey = OPRWebviewActivity.getPrivateKeyFromCookie(); + final String name = OPRWebviewActivity.getUsernameFromCookie(); + if (privateKey == null || privateKey.isEmpty()) { + OprStartFragment.showInstance(mapActivity.getSupportFragmentManager()); + return; + } + new Thread(new Runnable() { + @Override + public void run() { + if (openDBAPI.checkPrivateKeyValid(name,privateKey)){ + app.runInUIThread(new Runnable() { + @Override + public void run() { + Intent intent = new Intent(); + intent.setType("image/*"); + intent.setAction(Intent.ACTION_GET_CONTENT); + mapActivity.startActivityForResult(Intent.createChooser(intent, + mapActivity.getString(R.string.select_picture)), PICK_IMAGE); + } + }); + } + else { + OprStartFragment.showInstance(mapActivity.getSupportFragmentManager()); + } + } + }).start(); + } + + private void handleSelectedImage(final View view, final InputStream image) { + Thread t = new Thread(new Runnable() { + @Override + public void run() { + try { + uploadImageToPlace(view, image); + } catch (Exception e) { + LOG.error(e); + } + } + }); + t.start(); + } + + private void uploadImageToPlace(View view, InputStream image) { + //compress image + BufferedInputStream bufferedInputStream = new BufferedInputStream(image); + Bitmap bmp = BitmapFactory.decodeStream(bufferedInputStream); + ByteArrayOutputStream os = new ByteArrayOutputStream(); + bmp.compress(Bitmap.CompressFormat.PNG, 70, os); + byte[] buff = os.toByteArray(); + InputStream serverData = new ByteArrayInputStream(buff); + String url = BuildConfig.OPR_BASE_URL + "api/ipfs/image"; + String response = NetworkUtils.sendPostDataRequest(url, serverData); + if (response != null) { + int res = 0; + try { + res = openDBAPI.uploadImage( + placeId, + OPRWebviewActivity.getPrivateKeyFromCookie() +, OPRWebviewActivity.getUsernameFromCookie(), + response); + } catch (FailedVerificationException e) { + LOG.error(e); + app.showToastMessage(view.getResources().getString(R.string.cannot_upload_image)); + } + if (res != 200) { + //image was uploaded but not added to blockchain + app.showToastMessage(view.getResources().getString(R.string.cannot_upload_image)); + } else { + String str = MessageFormat.format(view.getResources() + .getString(R.string.successfully_uploaded_pattern), 1, 1); + app.showToastMessage(str); + //refresh the image + startLoadingImagesTask(); + } + } else { + app.showToastMessage(view.getResources().getString(R.string.cannot_upload_image)); + } + } + }); //TODO feature under development - b.setVisibility(View.GONE); + //b.setVisibility(View.GONE); b.setTextColor(ContextCompat.getColor(context, R.color.preference_category_title)); return b; } @@ -357,6 +473,10 @@ public class MenuBuilder { } onlinePhotoCards = new ArrayList<>(); onlinePhotoCardsRow.setProgressCard(); + startLoadingImagesTask(); + } + + private void startLoadingImagesTask(){ execute(new GetImageCardsTask(mapActivity, getLatLon(), getAdditionalCardParams(), new GetImageCardsListener() { @Override @@ -364,6 +484,11 @@ public class MenuBuilder { processOnlinePhotosCards(cardList); } + @Override + public void onPlaceIdAcquired(String[] placeId) { + MenuBuilder.this.placeId = placeId; + } + @Override public void onFinish(List cardList) { if (!isHidden()) { diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java index 1108692934..33feb6c5a3 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/builders/cards/ImageCard.java @@ -438,6 +438,8 @@ public abstract class ImageCard extends AbstractCard { public interface GetImageCardsListener { void onPostProcess(List cardList); + void onPlaceIdAcquired(String[] placeId); + void onFinish(List cardList); } @@ -464,8 +466,7 @@ public abstract class ImageCard extends AbstractCard { if (response != null) { getPicturesForPlace(result, response); String[] id = getIdFromResponse(response); - //TODO perform something with image - //listener.onOPRPlaceIdAcquired(id); + listener.onPlaceIdAcquired(id); } } try { diff --git a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java index 0d77c49d25..fbaa401cf6 100644 --- a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java +++ b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java @@ -28,7 +28,7 @@ import static org.openplacereviews.opendb.SecUtils.*; public class OpenDBAPI { private static final Log log = PlatformUtil.getLog(SecUtils.class); private static final String checkLoginEndpoint = "api/auth/user-check-loginkey?"; - private static final String LOGIN_SUCCESS_MESSAGE = "success"; + private static final String LOGIN_SUCCESS_MESSAGE = "{\"result\":\"OK\"}"; private static final int THREAD_ID = 11200; /* @@ -39,7 +39,7 @@ public class OpenDBAPI { * Need to encode key * Do not call on mainThread */ - public boolean checkPrivateKeyValid(String username, String privateKey) throws UnsupportedEncodingException { + public boolean checkPrivateKeyValid(String username, String privateKey) { String url = null; try { url = BuildConfig.OPR_BASE_URL + checkLoginEndpoint + @@ -50,7 +50,7 @@ public class OpenDBAPI { //need to encode the key URLEncoder.encode(privateKey, "UTF-8"); } catch (UnsupportedEncodingException e) { - throw e; + return false; } StringBuilder response = new StringBuilder(); return (NetworkUtils.sendGetRequest(url,null,response) == null) && @@ -63,7 +63,7 @@ public class OpenDBAPI { Security.addProvider(new BouncyCastleProvider()); } KeyPair kp = SecUtils.getKeyPair(ALGO_EC, privateKey, null); - String signed = username;// + ":opr-web"; + String signed = username + ":opr-web"; JsonFormatter formatter = new JsonFormatter(); OPRImage OPRImage = new GsonBuilder().create().fromJson(image, OPRImage.class); From a2207239904c016a8f6d9f12082b22bf2e632434 Mon Sep 17 00:00:00 2001 From: Simon Yakymovych Date: Fri, 20 Nov 2020 14:13:10 +0200 Subject: [PATCH 08/11] ui bug fix --- OsmAnd/res/layout/fragment_opr_login.xml | 171 ++++++++++-------- .../plus/mapcontextmenu/MenuBuilder.java | 60 +++--- .../net/osmand/plus/osmedit/opr/OPRImage.java | 8 - .../osmand/plus/osmedit/opr/OpenDBAPI.java | 17 +- 4 files changed, 137 insertions(+), 119 deletions(-) delete mode 100644 OsmAnd/src/net/osmand/plus/osmedit/opr/OPRImage.java diff --git a/OsmAnd/res/layout/fragment_opr_login.xml b/OsmAnd/res/layout/fragment_opr_login.xml index 3313679aec..7bd292e977 100644 --- a/OsmAnd/res/layout/fragment_opr_login.xml +++ b/OsmAnd/res/layout/fragment_opr_login.xml @@ -1,95 +1,114 @@ - + + + + + + + + + + + + + + + + + + + + + + + + + + - - - - - - - - - - - - - + android:gravity="bottom" + android:orientation="vertical"> + android:text="@string/register_opr_create_new_account" + android:textColor="@color/color_white" /> + android:layout_height="wrap_content" + android:layout_gravity="bottom" + android:layout_marginLeft="@dimen/content_padding" + android:layout_marginTop="@dimen/dashPadding" + android:layout_marginRight="@dimen/content_padding" + android:background="@color/activity_background_color_light" + android:text="@string/register_opr_have_account" + android:textColor="@color/icon_color_active_light" /> - \ No newline at end of file diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java index 30b7f05cd8..20db3e7d74 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java @@ -112,6 +112,32 @@ public class MenuBuilder { private View view; private OpenDBAPI openDBAPI = new OpenDBAPI(); private String[] placeId = new String[0]; + private GetImageCardsListener imageCardListener = new GetImageCardsListener() { + @Override + public void onPostProcess(List cardList) { + processOnlinePhotosCards(cardList); + } + + @Override + public void onPlaceIdAcquired(String[] placeId) { + MenuBuilder.this.placeId = placeId; + } + + @Override + public void onFinish(List cardList) { + if (!isHidden()) { + List cards = new ArrayList<>(); + cards.addAll(cardList); + if (cardList.size() == 0) { + cards.add(new NoImagesCard(mapActivity)); + } + if (onlinePhotoCardsRow != null) { + onlinePhotoCardsRow.setCards(cards); + } + onlinePhotoCards = cards; + } + } + }; public interface CollapseExpandListener { void onCollapseExpand(boolean collapsed); @@ -445,7 +471,7 @@ public class MenuBuilder { .getString(R.string.successfully_uploaded_pattern), 1, 1); app.showToastMessage(str); //refresh the image - startLoadingImagesTask(); + execute(new GetImageCardsTask(mapActivity, getLatLon(), getAdditionalCardParams(), imageCardListener)); } } else { app.showToastMessage(view.getResources().getString(R.string.cannot_upload_image)); @@ -471,39 +497,13 @@ public class MenuBuilder { if (onlinePhotoCardsRow == null) { return; } - onlinePhotoCards = new ArrayList<>(); - onlinePhotoCardsRow.setProgressCard(); startLoadingImagesTask(); } private void startLoadingImagesTask(){ - execute(new GetImageCardsTask(mapActivity, getLatLon(), getAdditionalCardParams(), - new GetImageCardsListener() { - @Override - public void onPostProcess(List cardList) { - processOnlinePhotosCards(cardList); - } - - @Override - public void onPlaceIdAcquired(String[] placeId) { - MenuBuilder.this.placeId = placeId; - } - - @Override - public void onFinish(List cardList) { - if (!isHidden()) { - List cards = new ArrayList<>(); - cards.addAll(cardList); - if (cardList.size() == 0) { - cards.add(new NoImagesCard(mapActivity)); - } - if (onlinePhotoCardsRow != null) { - onlinePhotoCardsRow.setCards(cards); - } - onlinePhotoCards = cards; - } - } - })); + onlinePhotoCards = new ArrayList<>(); + onlinePhotoCardsRow.setProgressCard(); + execute(new GetImageCardsTask(mapActivity, getLatLon(), getAdditionalCardParams(), imageCardListener)); } protected Map getAdditionalCardParams() { diff --git a/OsmAnd/src/net/osmand/plus/osmedit/opr/OPRImage.java b/OsmAnd/src/net/osmand/plus/osmedit/opr/OPRImage.java deleted file mode 100644 index dfffbf9973..0000000000 --- a/OsmAnd/src/net/osmand/plus/osmedit/opr/OPRImage.java +++ /dev/null @@ -1,8 +0,0 @@ -package net.osmand.plus.osmedit.opr; - -public class OPRImage { - public String type; - public String hash; - public String cid; - public String extension; -} \ No newline at end of file diff --git a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java index fbaa401cf6..ad19add249 100644 --- a/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java +++ b/OsmAnd/src/net/osmand/plus/osmedit/opr/OpenDBAPI.java @@ -66,17 +66,17 @@ public class OpenDBAPI { String signed = username + ":opr-web"; JsonFormatter formatter = new JsonFormatter(); - OPRImage OPRImage = new GsonBuilder().create().fromJson(image, OPRImage.class); + OPRImage oprImage = new GsonBuilder().create().fromJson(image, OPRImage.class); OpOperation opOperation = new OpOperation(); opOperation.setType("opr.place"); List edits = new ArrayList<>(); Map edit = new TreeMap<>(); List imageResponseList = new ArrayList<>(); Map imageMap = new TreeMap<>(); - imageMap.put("cid", OPRImage.cid); - imageMap.put("hash", OPRImage.hash); - imageMap.put("extension", OPRImage.extension); - imageMap.put("type", OPRImage.type); + imageMap.put("cid", oprImage.cid); + imageMap.put("hash", oprImage.hash); + imageMap.put("extension", oprImage.extension); + imageMap.put("type", oprImage.type); imageResponseList.add(imageMap); List ids = new ArrayList<>(Arrays.asList(placeId)); Map change = new TreeMap<>(); @@ -131,4 +131,11 @@ public class OpenDBAPI { } return -1; } + + public class OPRImage { + public String type; + public String hash; + public String cid; + public String extension; + } } \ No newline at end of file From 5a91901eeefca5c4176940b423f2155dc358a0b2 Mon Sep 17 00:00:00 2001 From: simon Date: Mon, 23 Nov 2020 11:02:02 +0200 Subject: [PATCH 09/11] conflict resolved --- OsmAnd/res/layout/fragment_opr_login.xml | 181 +++++++++--------- OsmAnd/res/values/strings.xml | 29 ++- .../plus/mapcontextmenu/MenuBuilder.java | 150 ++++++++------- .../openplacereviews/OprStartFragment.java | 3 - 4 files changed, 183 insertions(+), 180 deletions(-) diff --git a/OsmAnd/res/layout/fragment_opr_login.xml b/OsmAnd/res/layout/fragment_opr_login.xml index b8877dc710..e941075f54 100644 --- a/OsmAnd/res/layout/fragment_opr_login.xml +++ b/OsmAnd/res/layout/fragment_opr_login.xml @@ -1,114 +1,105 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + android:layout_height="@dimen/dialog_button_height" + android:layout_marginBottom="@dimen/content_padding_small" /> + android:layout_height="@dimen/dialog_button_height" /> - \ No newline at end of file + + \ No newline at end of file diff --git a/OsmAnd/res/values/strings.xml b/OsmAnd/res/values/strings.xml index aaa8241710..72fb10e393 100644 --- a/OsmAnd/res/values/strings.xml +++ b/OsmAnd/res/values/strings.xml @@ -11,6 +11,19 @@ Thx - Hardy --> + OsmAnd shows photos from several sources:\nOpenPlaceReviews - POI photos;\nMapillary - street-level imagery;\nWeb / Wikimedia - POI photos specified in OpenStreetMap data. + Use dev.openstreetmap.org + Switch to use "dev.openstreetmap.org" instead of "openstreetmap.org" to testing uploading OSM Note / POI / GPX. + Add to OpenPlaceReviews + Add to Mapillary + Select items that will be imported. + Select groups that will be imported. + There is not enough space + Your device only has %1$s free. Please free up some space or unselect some items to export. + Needed for import + Select the data to be exported to the file. + Approximate file size + Resources Select picture Cannot upload image, please, try again later Motorboat @@ -18,10 +31,10 @@ Search history I already have an account Create new account - Log in on the open data project website OpenPlaceReviews.org to upload even more photos. + Photos are provided by open data project OpenPlaceReviews.org. In order to upload your photos you need to sign up on website. Register on\nOpenPlaceReviews.org Add photo - Log in using the safe OAuth method or use your username and password. + You can log in using the safe OAuth method or use your login and password. Comment OSM Note Close OSM Note \"Trackable\" means the trace does not show up in any public listings, but processed trackpoints with timestamps from it (that can\'t be associated with you directly) do through downloads from the public GPS API. @@ -36,15 +49,15 @@ OsmAnd Live subscription has been expired There is a problem with your subscription. Click the button to go to the Google Play subscription settings to fix your payment method. Manage subscription - Username + Login Password Account - Log in with username and password - Log in to upload new or modified changes,\n\neither with OAuth or using your username and password. + Use login and password + You need to login to upload new or modified changes. \n\nYou can log in using the safe OAuth method or use your login and password. You can view all your not yet uploaded edits or OSM bugs in %1$s. Uploaded points don’t show in OsmAnd. - Log in with OpenStreetMap - Login for OpenStreetMap.org - Login for OpenStreetMap + Sign in with OpenStreetMap + Login to OpenStreetMap.org + Login to OpenStreetMap These plugin setting are global, and apply to all profiles You need to add at least two points Travel diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java index 4d8ff2d548..8a82f4d96b 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java @@ -51,6 +51,7 @@ import net.osmand.plus.mapcontextmenu.builders.cards.ImageCard; import net.osmand.plus.mapcontextmenu.builders.cards.ImageCard.GetImageCardsTask; import net.osmand.plus.mapcontextmenu.builders.cards.NoImagesCard; import net.osmand.plus.mapcontextmenu.controllers.TransportStopController; +import net.osmand.plus.openplacereviews.AddPhotosBottomSheetDialogFragment; import net.osmand.plus.openplacereviews.OPRWebviewActivity; import net.osmand.plus.openplacereviews.OprStartFragment; import net.osmand.plus.osmedit.opr.OpenDBAPI; @@ -78,6 +79,8 @@ import static net.osmand.plus.mapcontextmenu.builders.cards.ImageCard.GetImageCa public class MenuBuilder { + private static final int PICK_IMAGE = 1231; + private static final Log LOG = PlatformUtil.getLog(MenuBuilder.class); public static final float SHADOW_HEIGHT_TOP_DP = 17f; public static final int TITLE_LIMIT = 60; protected static final String[] arrowChars = new String[] {"=>", " - "}; @@ -107,10 +110,7 @@ public class MenuBuilder { private String preferredMapAppLang; private boolean transliterateNames; - private static final int PICK_IMAGE = 1231; - private static final Log LOG = PlatformUtil.getLog(MenuBuilder.class); - private View view; - private OpenDBAPI openDBAPI = new OpenDBAPI(); + private final OpenDBAPI openDBAPI = new OpenDBAPI(); private String[] placeId = new String[0]; private GetImageCardsListener imageCardListener = new GetImageCardsListener() { @Override @@ -126,8 +126,7 @@ public class MenuBuilder { @Override public void onFinish(List cardList) { if (!isHidden()) { - List cards = new ArrayList<>(); - cards.addAll(cardList); + List cards = new ArrayList(cardList); if (cardList.size() == 0) { cards.add(new NoImagesCard(mapActivity)); } @@ -386,19 +385,7 @@ public class MenuBuilder { b.setOnClickListener(new OnClickListener() { @Override public void onClick(final View view) { - mapActivity.registerActivityResultListener(new ActivityResultListener(PICK_IMAGE, - new ActivityResultListener.OnActivityResultListener() { - @Override - public void onResult(int resultCode, Intent resultData) { - InputStream inputStream = null; - try { - inputStream = mapActivity.getContentResolver().openInputStream(resultData.getData()); - } catch (Exception e) { - LOG.error(e); - } - handleSelectedImage(view, inputStream); - } - })); + registerResultListener(view); final String privateKey = OPRWebviewActivity.getPrivateKeyFromCookie(); final String name = OPRWebviewActivity.getUsernameFromCookie(); if (privateKey == null || privateKey.isEmpty()) { @@ -408,7 +395,7 @@ public class MenuBuilder { new Thread(new Runnable() { @Override public void run() { - if (openDBAPI.checkPrivateKeyValid(name,privateKey)){ + if (openDBAPI.checkPrivateKeyValid(name, privateKey)) { app.runInUIThread(new Runnable() { @Override public void run() { @@ -419,64 +406,12 @@ public class MenuBuilder { mapActivity.getString(R.string.select_picture)), PICK_IMAGE); } }); - } - else { + } else { OprStartFragment.showInstance(mapActivity.getSupportFragmentManager()); } } }).start(); } - - private void handleSelectedImage(final View view, final InputStream image) { - Thread t = new Thread(new Runnable() { - @Override - public void run() { - try { - uploadImageToPlace(view, image); - } catch (Exception e) { - LOG.error(e); - } - } - }); - t.start(); - } - - private void uploadImageToPlace(View view, InputStream image) { - //compress image - BufferedInputStream bufferedInputStream = new BufferedInputStream(image); - Bitmap bmp = BitmapFactory.decodeStream(bufferedInputStream); - ByteArrayOutputStream os = new ByteArrayOutputStream(); - bmp.compress(Bitmap.CompressFormat.PNG, 70, os); - byte[] buff = os.toByteArray(); - InputStream serverData = new ByteArrayInputStream(buff); - String url = BuildConfig.OPR_BASE_URL + "api/ipfs/image"; - String response = NetworkUtils.sendPostDataRequest(url, serverData); - if (response != null) { - int res = 0; - try { - res = openDBAPI.uploadImage( - placeId, - OPRWebviewActivity.getPrivateKeyFromCookie() -, OPRWebviewActivity.getUsernameFromCookie(), - response); - } catch (FailedVerificationException e) { - LOG.error(e); - app.showToastMessage(view.getResources().getString(R.string.cannot_upload_image)); - } - if (res != 200) { - //image was uploaded but not added to blockchain - app.showToastMessage(view.getResources().getString(R.string.cannot_upload_image)); - } else { - String str = MessageFormat.format(view.getResources() - .getString(R.string.successfully_uploaded_pattern), 1, 1); - app.showToastMessage(str); - //refresh the image - execute(new GetImageCardsTask(mapActivity, getLatLon(), getAdditionalCardParams(), imageCardListener)); - } - } else { - app.showToastMessage(view.getResources().getString(R.string.cannot_upload_image)); - } - } }); //TODO feature under development //b.setVisibility(View.GONE); @@ -493,6 +428,73 @@ public class MenuBuilder { false, null, false); } + private void registerResultListener(final View view) { + mapActivity.registerActivityResultListener(new ActivityResultListener(PICK_IMAGE, new ActivityResultListener. + OnActivityResultListener() { + @Override + public void onResult(int resultCode, Intent resultData) { + handleSelectedImage(view, resultData.getData()); + } + })); + } + + private void handleSelectedImage(final View view, final Uri uri) { + Thread t = new Thread(new Runnable() { + @Override + public void run() { + InputStream inputStream = null; + try { + inputStream = app.getContentResolver().openInputStream(uri); + if (inputStream != null) { + uploadImageToPlace(view, inputStream); + } + } catch (Exception e) { + LOG.error(e); + } finally { + Algorithms.closeStream(inputStream); + } + } + }); + t.start(); + } + + private void uploadImageToPlace(View view, InputStream image) { + InputStream serverData = new ByteArrayInputStream(compressImage(image)); + String url = BuildConfig.OPR_BASE_URL + "api/ipfs/image"; + String response = NetworkUtils.sendPostDataRequest(url, serverData); + if (response != null) { + int res = 0; + try { + res = openDBAPI.uploadImage( + placeId, + OPRWebviewActivity.getPrivateKeyFromCookie(), + OPRWebviewActivity.getUsernameFromCookie(), + response); + } catch (FailedVerificationException e) { + LOG.error(e); + app.showToastMessage(R.string.cannot_upload_image); + } + if (res != 200) { + //image was uploaded but not added to blockchain + app.showToastMessage(R.string.cannot_upload_image); + } else { + app.showToastMessage(R.string.successfully_uploaded_pattern, 1, 1); + //refresh the image + execute(new GetImageCardsTask(mapActivity, getLatLon(), getAdditionalCardParams(), imageCardListener)); + } + } else { + app.showToastMessage(R.string.cannot_upload_image); + } + } + + private byte[] compressImage(InputStream image) { + BufferedInputStream bufferedInputStream = new BufferedInputStream(image); + Bitmap bmp = BitmapFactory.decodeStream(bufferedInputStream); + ByteArrayOutputStream os = new ByteArrayOutputStream(); + bmp.compress(Bitmap.CompressFormat.PNG, 70, os); + return os.toByteArray(); + } + private void startLoadingImages() { if (onlinePhotoCardsRow == null) { return; @@ -500,7 +502,7 @@ public class MenuBuilder { startLoadingImagesTask(); } - private void startLoadingImagesTask(){ + private void startLoadingImagesTask() { onlinePhotoCards = new ArrayList<>(); onlinePhotoCardsRow.setProgressCard(); execute(new GetImageCardsTask(mapActivity, getLatLon(), getAdditionalCardParams(), imageCardListener)); diff --git a/OsmAnd/src/net/osmand/plus/openplacereviews/OprStartFragment.java b/OsmAnd/src/net/osmand/plus/openplacereviews/OprStartFragment.java index 19a4ba57be..2f936cf21e 100644 --- a/OsmAnd/src/net/osmand/plus/openplacereviews/OprStartFragment.java +++ b/OsmAnd/src/net/osmand/plus/openplacereviews/OprStartFragment.java @@ -12,17 +12,14 @@ import android.view.LayoutInflater; import android.view.View; import android.view.ViewGroup; import android.widget.TextView; - import androidx.annotation.NonNull; import androidx.annotation.Nullable; import androidx.fragment.app.FragmentActivity; import androidx.fragment.app.FragmentManager; - import net.osmand.PlatformUtil; import net.osmand.plus.R; import net.osmand.plus.UiUtilities; import net.osmand.plus.base.BaseOsmAndFragment; - import org.apache.commons.logging.Log; public class OprStartFragment extends BaseOsmAndFragment { From 0462f635a8ebf04e5f8b64a9697eedad6481149a Mon Sep 17 00:00:00 2001 From: simon Date: Mon, 23 Nov 2020 11:17:18 +0200 Subject: [PATCH 10/11] visibility updated --- OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java index 8a82f4d96b..b0a5897ceb 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java @@ -414,7 +414,7 @@ public class MenuBuilder { } }); //TODO feature under development - //b.setVisibility(View.GONE); + b.setVisibility(View.GONE); b.setTextColor(ContextCompat.getColor(context, R.color.preference_category_title)); return b; } From c7f7a9efdca4a93e2a1054f0c751242a8a4144ff Mon Sep 17 00:00:00 2001 From: simon Date: Mon, 23 Nov 2020 15:49:19 +0200 Subject: [PATCH 11/11] condition added --- .../plus/mapcontextmenu/MenuBuilder.java | 62 +++++++++---------- 1 file changed, 30 insertions(+), 32 deletions(-) diff --git a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java index b0a5897ceb..0a6fa296d0 100644 --- a/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java +++ b/OsmAnd/src/net/osmand/plus/mapcontextmenu/MenuBuilder.java @@ -6,11 +6,7 @@ import android.content.Context; import android.content.DialogInterface; import android.content.Intent; import android.content.res.ColorStateList; -import android.graphics.Bitmap; -import android.graphics.BitmapFactory; -import android.graphics.Color; -import android.graphics.PorterDuff; -import android.graphics.Typeface; +import android.graphics.*; import android.graphics.drawable.Drawable; import android.graphics.drawable.GradientDrawable; import android.net.Uri; @@ -64,7 +60,6 @@ import net.osmand.plus.widgets.TextViewEx; import net.osmand.plus.widgets.tools.ClickableSpanTouchListener; import net.osmand.util.Algorithms; import net.osmand.util.MapUtils; - import org.apache.commons.logging.Log; import org.openplacereviews.opendb.util.exception.FailedVerificationException; @@ -72,7 +67,6 @@ import java.io.BufferedInputStream; import java.io.ByteArrayInputStream; import java.io.ByteArrayOutputStream; import java.io.InputStream; -import java.text.MessageFormat; import java.util.*; import static net.osmand.plus.mapcontextmenu.builders.cards.ImageCard.GetImageCardsTask.GetImageCardsListener; @@ -385,32 +379,36 @@ public class MenuBuilder { b.setOnClickListener(new OnClickListener() { @Override public void onClick(final View view) { - registerResultListener(view); - final String privateKey = OPRWebviewActivity.getPrivateKeyFromCookie(); - final String name = OPRWebviewActivity.getUsernameFromCookie(); - if (privateKey == null || privateKey.isEmpty()) { - OprStartFragment.showInstance(mapActivity.getSupportFragmentManager()); - return; - } - new Thread(new Runnable() { - @Override - public void run() { - if (openDBAPI.checkPrivateKeyValid(name, privateKey)) { - app.runInUIThread(new Runnable() { - @Override - public void run() { - Intent intent = new Intent(); - intent.setType("image/*"); - intent.setAction(Intent.ACTION_GET_CONTENT); - mapActivity.startActivityForResult(Intent.createChooser(intent, - mapActivity.getString(R.string.select_picture)), PICK_IMAGE); - } - }); - } else { - OprStartFragment.showInstance(mapActivity.getSupportFragmentManager()); - } + if (false) { + AddPhotosBottomSheetDialogFragment.showInstance(mapActivity.getSupportFragmentManager()); + } else { + registerResultListener(view); + final String privateKey = OPRWebviewActivity.getPrivateKeyFromCookie(); + final String name = OPRWebviewActivity.getUsernameFromCookie(); + if (privateKey == null || privateKey.isEmpty()) { + OprStartFragment.showInstance(mapActivity.getSupportFragmentManager()); + return; } - }).start(); + new Thread(new Runnable() { + @Override + public void run() { + if (openDBAPI.checkPrivateKeyValid(name, privateKey)) { + app.runInUIThread(new Runnable() { + @Override + public void run() { + Intent intent = new Intent(); + intent.setType("image/*"); + intent.setAction(Intent.ACTION_GET_CONTENT); + mapActivity.startActivityForResult(Intent.createChooser(intent, + mapActivity.getString(R.string.select_picture)), PICK_IMAGE); + } + }); + } else { + OprStartFragment.showInstance(mapActivity.getSupportFragmentManager()); + } + } + }).start(); + } } }); //TODO feature under development